irods / irods_resource_plugin_rados

Cacheless Ceph/rados resource plugin for iRODS
BSD 3-Clause "New" or "Revised" License
7 stars 6 forks source link

Compile error with IRADOS_DEBUG #10

Closed joergsteinkamp closed 6 years ago

joergsteinkamp commented 6 years ago

When DEBUG messages are turned on by defining IRADOS_DEBUG I get a compile time error. Solved by changing as follows: Line 650 in libirados.cpp should be "blob_oid.str().c_str()" instead of "blob_oid.c_str()" in function int get_next_fd().

trel commented 6 years ago

like this?


diff --git a/irados/libirados.cpp b/irados/libirados.cpp                                                                                                                             
index 8fbf24e..b1c0263 100644                                                                                                                                                        
--- a/irados/libirados.cpp                                                                                                                                                           
+++ b/irados/libirados.cpp                                                                                                                                                           
@@ -647,7 +647,7 @@ int get_next_fd() {                                                                                                                                              
 #ifdef IRADOS_DEBUG                                                                                                                                                                 
             rodsLog(LOG_NOTICE, "IRADOS_DEBUG %s ReadPart: from %s blob_id: %d, blob_offset: %lu, read: %lu, rados_read status: %d, (fd: %d)",                                      
                     __func__,                                                                                                                                                       
-                    blob_oid.c_str(),                                                                                                                                               
+                    blob_oid.str().c_str(),                                                                                                                                         
                     blob_id,                                                                                                                                                        
                     blob_offset,                                                                                                                                                    
                     read_len,                                                                                                                                                       ```
joergsteinkamp commented 6 years ago

Yes, that's it.