don't add wxVSCROLL if only wxHSCROLL is explicitly specified (and vice versa), only...

[wxWidgets.git] / src / unix / dlunix.cpp

diff --git a/src/unix/dlunix.cpp b/src/unix/dlunix.cpp
index 7e78ccb774d2e93ce224f6efa49900e79a0ee074..efe3f7e2ef2dfe5c6e782982fdaa80867eaea8b6 100644 (file)
--- a/src/unix/dlunix.cpp
+++ b/src/unix/dlunix.cpp
@@ -294,13 +294,13 @@ class wxDynamicLibraryDetailsCreator
 public:
     // create a new wxDynamicLibraryDetails from the given data
     static wxDynamicLibraryDetails *
 public:
     // create a new wxDynamicLibraryDetails from the given data
     static wxDynamicLibraryDetails *

-    New(unsigned long start, unsigned long end, const wxString& path)
+    New(void *start, void *end, const wxString& path)

     {
         wxDynamicLibraryDetails *details = new wxDynamicLibraryDetails;
         details->m_path = path;
         details->m_name = path.AfterLast(_T('/'));
     {
         wxDynamicLibraryDetails *details = new wxDynamicLibraryDetails;
         details->m_path = path;
         details->m_name = path.AfterLast(_T('/'));

-        details->m_address = wxUIntToPtr(start);
-        details->m_length = end - start;
+        details->m_address = start;
+        details->m_length = (char *)end - (char *)start;

 
         // try to extract the library version from its name
         const size_t posExt = path.rfind(_T(".so"));
 
         // try to extract the library version from its name
         const size_t posExt = path.rfind(_T(".so"));


@@ -339,16 +339,17 @@ wxDynamicLibraryDetailsArray wxDynamicLibrary::ListLoaded()
     {
         // details of the module currently being parsed
         wxString pathCur;
     {
         // details of the module currently being parsed
         wxString pathCur;

-        unsigned long startCur = 0,
-                      endCur = 0;
+        void *startCur = NULL,
+             *endCur = NULL;

 
         char path[1024];
         char buf[1024];
         while ( fgets(buf, WXSIZEOF(buf), file.fp()) )
         {
 
         char path[1024];
         char buf[1024];
         while ( fgets(buf, WXSIZEOF(buf), file.fp()) )
         {

-            // format is: start-end perm something? maj:min inode path
-            unsigned long start, end;
-            switch ( sscanf(buf, "%08lx-%08lx %*4s %*08x %*02d:%*02d %*d %1024s\n",
+            // format is: "start-end perm offset maj:min inode path", see proc(5)
+            void *start,
+                 *end;
+            switch ( sscanf(buf, "%p-%p %*4s %*p %*02x:%*02x %*d %1024s\n",

                             &start, &end, path) )
             {
                 case 2:
                             &start, &end, path) )
             {
                 case 2:


@@ -368,6 +369,9 @@ wxDynamicLibraryDetailsArray wxDynamicLibrary::ListLoaded()
                     continue;
             }
 
                     continue;
             }
 

+            wxASSERT_MSG( start >= endCur,
+                          _T("overlapping regions in /proc/self/maps?") );
+

             wxString pathNew = wxString::FromAscii(path);
             if ( pathCur.empty() )
             {
             wxString pathNew = wxString::FromAscii(path);
             if ( pathCur.empty() )
             {


@@ -376,10 +380,9 @@ wxDynamicLibraryDetailsArray wxDynamicLibrary::ListLoaded()
                 startCur = start;
                 endCur = end;
             }
                 startCur = start;
                 endCur = end;
             }

-            else if ( pathCur == pathNew )
+            else if ( pathCur == pathNew && endCur == end )

             {
             {

-                // continuation of the same module
-                wxASSERT_MSG( start == endCur, _T("hole in /proc/self/maps?") );
+                // continuation of the same module in the address space

                 endCur = end;
             }
             else // end of the current module
                 endCur = end;
             }
             else // end of the current module