Changeset 21034

Show
Ignore:
Timestamp:
02/21/08 17:41:26 (6 months ago)
Author:
patrick
Message:

De-inline constructors as they can be deceptively expensive to have as inline
code.

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • juggler/trunk/modules/vapor/vpr/md/NSPR/DynLoad/LibraryNSPR.cpp

    r20974 r21034  
    3636#include <vpr/vprConfig.h> 
    3737 
    38 #include <stdlib.h
     38#include <cstdlib
    3939#include <sstream> 
    4040 
     
    4949namespace vpr 
    5050{ 
     51 
     52LibraryNSPR::LibraryNSPR(const std::string& name) 
     53   : mName(name) 
     54   , mLibrary(NULL) 
     55{ 
     56   ; 
     57} 
     58 
     59LibraryNSPR::LibraryNSPR() 
     60   : mName("") 
     61   , mLibrary(NULL) 
     62{ 
     63   ; 
     64} 
     65 
     66LibraryNSPR::LibraryNSPR(const LibraryNSPR& lib) 
     67   : mName("") 
     68   , mLibrary(NULL) 
     69{ 
     70   copy(lib); 
     71} 
    5172 
    5273LibraryNSPR::~LibraryNSPR() 
  • juggler/trunk/modules/vapor/vpr/md/NSPR/DynLoad/LibraryNSPR.h

    r20974 r21034  
    3939#include <vpr/vprConfig.h> 
    4040 
    41 #include <stdlib.h
     41#include <cstdlib
    4242#include <string> 
    4343#include <prlink.h> 
     
    7171    *             The name may be an absolute path or a relative path. 
    7272    */ 
    73    LibraryNSPR(const std::string& name) 
    74       : mName(name) 
    75       , mLibrary(NULL) 
    76    { 
    77       ; 
    78    } 
     73   LibraryNSPR(const std::string& name); 
    7974 
    8075   /** 
     
    8782    * @see findSymbolAndLibrary, load 
    8883    */ 
    89    LibraryNSPR() 
    90       : mName("") 
    91       , mLibrary(NULL) 
    92    { 
    93       ; 
    94    } 
     84   LibraryNSPR(); 
    9585 
    9686   /** 
    9787    * Copy constructor. 
    9888    */ 
    99    LibraryNSPR(const LibraryNSPR& lib) 
    100       : mName("") 
    101       , mLibrary(NULL) 
    102    { 
    103       copy(lib); 
    104    } 
     89   LibraryNSPR(const LibraryNSPR& lib); 
    10590 
    10691   /** 
  • juggler/trunk/modules/vapor/vpr/md/POSIX/DynLoad/LibraryUNIX.cpp

    r20974 r21034  
    3737 
    3838#include <dlfcn.h> 
    39 #include <stdlib.h
     39#include <cstdlib
    4040#include <sstream> 
    4141#include <boost/concept_check.hpp> 
     
    5050namespace vpr 
    5151{ 
     52 
     53LibraryUNIX::LibraryUNIX(const std::string& name) 
     54   : mName(name) 
     55   , mLibrary(NULL) 
     56{ 
     57   ; 
     58} 
     59 
     60LibraryUNIX::LibraryUNIX() 
     61   : mName("") 
     62   , mLibrary(NULL) 
     63{ 
     64   ; 
     65} 
     66 
     67LibraryUNIX::LibraryUNIX(const LibraryUNIX& lib) 
     68   : mName("") 
     69   , mLibrary(NULL) 
     70{ 
     71   copy(lib); 
     72} 
    5273 
    5374LibraryUNIX::~LibraryUNIX() 
  • juggler/trunk/modules/vapor/vpr/md/POSIX/DynLoad/LibraryUNIX.h

    r20974 r21034  
    3939#include <vpr/vprConfig.h> 
    4040 
    41 #include <stdlib.h
     41#include <cstdlib
    4242#include <string> 
    4343 
     
    6969    *             The name may be an absolute path or a relative path. 
    7070    */ 
    71    LibraryUNIX(const std::string& name) 
    72       : mName(name) 
    73       , mLibrary(NULL) 
    74    { 
    75       ; 
    76    } 
     71   LibraryUNIX(const std::string& name); 
    7772 
    7873   /** 
     
    8580    * @see findSymbolAndLibrary, load 
    8681    */ 
    87    LibraryUNIX() 
    88       : mName("") 
    89       , mLibrary(NULL) 
    90    { 
    91       ; 
    92    } 
     82   LibraryUNIX(); 
    9383 
    9484   /** 
    9585    * Copy constructor. 
    9686    */ 
    97    LibraryUNIX(const LibraryUNIX& lib) 
    98       : mName("") 
    99       , mLibrary(NULL) 
    100    { 
    101       copy(lib); 
    102    } 
     87   LibraryUNIX(const LibraryUNIX& lib); 
    10388 
    10489   /**