[llvm-commits] CVS: llvm/lib/Bytecode/Archive/ArchiveReader.cpp ArchiveWriter.cpp

[Reid Spencer]

Changes in directory llvm/lib/Bytecode/Archive:

ArchiveReader.cpp updated: 1.40 -> 1.41
ArchiveWriter.cpp updated: 1.18 -> 1.19
---
Log message:

For PR495: http://llvm.cs.uiuc.edu/PR495 :
Get rid of the difference between file paths and directory paths. The Path
class now simply stores a path that can refer to either a file or a 
directory. This required various changes in the implementation and interface
of the class with the corresponding impact to its users. Doxygen comments were
also updated to reflect these changes. Interface changes are:

appendDirectory -> appendComponent
appendFile -> appendComponent
elideDirectory -> eraseComponent
elideFile -> eraseComponent
elideSuffix -> eraseSuffix
renameFile -> rename
setDirectory -> set
setFile -> set

Changes pass Dejagnu and llvm-test/SingleSource tests.


---
Diffs of the changes:  (+4 -4)

 ArchiveReader.cpp |    2 +-
 ArchiveWriter.cpp |    6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)


Index: llvm/lib/Bytecode/Archive/ArchiveReader.cpp
diff -u llvm/lib/Bytecode/Archive/ArchiveReader.cpp:1.40 llvm/lib/Bytecode/Archive/ArchiveReader.cpp:1.41
--- llvm/lib/Bytecode/Archive/ArchiveReader.cpp:1.40	Thu Apr 21 16:13:18 2005
+++ llvm/lib/Bytecode/Archive/ArchiveReader.cpp	Thu Jul  7 18:21:43 2005
@@ -187,7 +187,7 @@
   member->next = 0;
   member->prev = 0;
   member->parent = this;
-  member->path.setFile(pathname);
+  member->path.set(pathname);
   member->info.fileSize = MemberSize;
   member->info.modTime.fromEpochTime(atoi(Hdr->date));
   unsigned int mode;


Index: llvm/lib/Bytecode/Archive/ArchiveWriter.cpp
diff -u llvm/lib/Bytecode/Archive/ArchiveWriter.cpp:1.18 llvm/lib/Bytecode/Archive/ArchiveWriter.cpp:1.19
--- llvm/lib/Bytecode/Archive/ArchiveWriter.cpp:1.18	Thu Apr 21 16:13:18 2005
+++ llvm/lib/Bytecode/Archive/ArchiveWriter.cpp	Thu Jul  7 18:21:43 2005
@@ -450,17 +450,17 @@
       // Close up shop
       FinalFile.close();
       arch.close();
-      TmpArchive.destroyFile();
+      TmpArchive.destroy();
 
     } else {
       // We don't have to insert the symbol table, so just renaming the temp
       // file to the correct name will suffice.
-      TmpArchive.renameFile(archPath);
+      TmpArchive.rename(archPath);
     }
   } catch (...) {
     // Make sure we clean up.
     if (TmpArchive.exists())
-      TmpArchive.destroyFile();
+      TmpArchive.destroy();
     throw;
   }
 }