# HG changeset patch # User Sebastien Jodogne # Date 1771602934 -3600 # Node ID 4f5811d75978fc2bd9a84436d1ef0be1e88a9057 # Parent 24f00121619f9ce3d2c57b17dce036bc2ad92ff1 for filenames in ZipWriter, replace "char*" by "std::string" diff -r 24f00121619f -r 4f5811d75978 OrthancFramework/Sources/Compression/HierarchicalZipWriter.cpp --- a/OrthancFramework/Sources/Compression/HierarchicalZipWriter.cpp Fri Feb 20 16:47:30 2026 +0100 +++ b/OrthancFramework/Sources/Compression/HierarchicalZipWriter.cpp Fri Feb 20 16:55:34 2026 +0100 @@ -85,7 +85,7 @@ return result; } - std::string HierarchicalZipWriter::Index::EnsureUniqueFilename(const char* filename) + std::string HierarchicalZipWriter::Index::EnsureUniqueFilename(const std::string& filename) { std::string standardized = KeepAlphanumeric(filename); @@ -122,12 +122,12 @@ return stack_.size() == 1; } - std::string HierarchicalZipWriter::Index::OpenFile(const char* name) + std::string HierarchicalZipWriter::Index::OpenFile(const std::string& name) { return GetCurrentDirectoryPath() + EnsureUniqueFilename(name); } - void HierarchicalZipWriter::Index::OpenDirectory(const char* name) + void HierarchicalZipWriter::Index::OpenDirectory(const std::string& name) { std::string d = EnsureUniqueFilename(name); @@ -199,13 +199,13 @@ return writer_.IsAppendToExisting(); } - void HierarchicalZipWriter::OpenFile(const char* name) + void HierarchicalZipWriter::OpenFile(const std::string& name) { std::string p = indexer_.OpenFile(name); writer_.OpenFile(p.c_str()); } - void HierarchicalZipWriter::OpenDirectory(const char* name) + void HierarchicalZipWriter::OpenDirectory(const std::string& name) { indexer_.OpenDirectory(name); } @@ -220,12 +220,13 @@ return indexer_.GetCurrentDirectoryPath(); } - void HierarchicalZipWriter::Write(const void *data, size_t length) + void HierarchicalZipWriter::Write(const void *data, + size_t length) { writer_.Write(data, length); } - void HierarchicalZipWriter::Write(const std::string &data) + void HierarchicalZipWriter::Write(const std::string& data) { writer_.Write(data); } diff -r 24f00121619f -r 4f5811d75978 OrthancFramework/Sources/Compression/HierarchicalZipWriter.h --- a/OrthancFramework/Sources/Compression/HierarchicalZipWriter.h Fri Feb 20 16:47:30 2026 +0100 +++ b/OrthancFramework/Sources/Compression/HierarchicalZipWriter.h Fri Feb 20 16:55:34 2026 +0100 @@ -60,7 +60,7 @@ Stack stack_; - std::string EnsureUniqueFilename(const char* filename); + std::string EnsureUniqueFilename(const std::string& name); public: Index(); @@ -69,9 +69,9 @@ bool IsRoot() const; - std::string OpenFile(const char* name); + std::string OpenFile(const std::string& name); - void OpenDirectory(const char* name); + void OpenDirectory(const std::string& name); void CloseDirectory(); @@ -103,9 +103,9 @@ bool IsAppendToExisting() const; - void OpenFile(const char* name); + void OpenFile(const std::string& name); - void OpenDirectory(const char* name); + void OpenDirectory(const std::string& name); void CloseDirectory(); diff -r 24f00121619f -r 4f5811d75978 OrthancFramework/Sources/Compression/ZipWriter.cpp --- a/OrthancFramework/Sources/Compression/ZipWriter.cpp Fri Feb 20 16:47:30 2026 +0100 +++ b/OrthancFramework/Sources/Compression/ZipWriter.cpp Fri Feb 20 16:55:34 2026 +0100 @@ -603,7 +603,7 @@ return compressionLevel_; } - void ZipWriter::OpenFile(const char* filename) + void ZipWriter::OpenFile(const std::string& filename) { Open(); @@ -614,7 +614,7 @@ if (isZip64_) { - result = zipOpenNewFileInZip64(pimpl_->file_, filename, + result = zipOpenNewFileInZip64(pimpl_->file_, filename.c_str(), &zfi, NULL, 0, NULL, 0, @@ -624,7 +624,7 @@ } else { - result = zipOpenNewFileInZip(pimpl_->file_, filename, + result = zipOpenNewFileInZip(pimpl_->file_, filename.c_str(), &zfi, NULL, 0, NULL, 0, diff -r 24f00121619f -r 4f5811d75978 OrthancFramework/Sources/Compression/ZipWriter.h --- a/OrthancFramework/Sources/Compression/ZipWriter.h Fri Feb 20 16:47:30 2026 +0100 +++ b/OrthancFramework/Sources/Compression/ZipWriter.h Fri Feb 20 16:55:34 2026 +0100 @@ -164,9 +164,10 @@ const boost::filesystem::path& GetOutputPath() const; - void OpenFile(const char* filename); + void OpenFile(const std::string& filename); - void Write(const void* data, size_t length); + void Write(const void* data, + size_t length); void Write(const std::string& data);