Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Fix compile issue due to stale header definitions #126

Open
wants to merge 1 commit into from

2 participants

@mrpdaemon

It seems like the function names in ConfigTest.cc were changed but
the corresponding names in ConfigTest.hh weren't updated, which
caused a compilation failure with g++ 4.6.3.

@mrpdaemon mrpdaemon Fix compile issue due to stale header definitions
It seems like the function names in ConfigTest.cc were changed but
the corresponding names in ConfigTest.hh weren't updated, which
caused a compilation failure with g++ 4.6.3.
a145c20
@marcusmueller

I can confirm that it fixes the compilation issue #159 AND removes the consequent linker failure resulting from the stle definitions.

Good work, merge fast.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Aug 26, 2012
  1. @mrpdaemon

    Fix compile issue due to stale header definitions

    mrpdaemon authored
    It seems like the function names in ConfigTest.cc were changed but
    the corresponding names in ConfigTest.hh weren't updated, which
    caused a compilation failure with g++ 4.6.3.
This page is out of date. Refresh to see the latest.
Showing with 4 additions and 6 deletions.
  1. +4 −6 libgrive/test/util/ConfigTest.hh
View
10 libgrive/test/util/ConfigTest.hh
@@ -31,15 +31,13 @@ public :
ConfigTest( ) ;
CPPUNIT_TEST_SUITE( ConfigTest ) ;
- CPPUNIT_TEST( TestInitialiseWithEmptyString ) ;
- CPPUNIT_TEST( TestInitialiseWithString ) ;
- CPPUNIT_TEST( TestInitialiseWithFileSystemPath ) ;
+ CPPUNIT_TEST( TestInitialiseWithNoPath ) ;
+ CPPUNIT_TEST( TestInitialiseWithPath ) ;
CPPUNIT_TEST_SUITE_END();
private :
- void TestInitialiseWithEmptyString( );
- void TestInitialiseWithString( );
- void TestInitialiseWithFileSystemPath( );
+ void TestInitialiseWithNoPath( );
+ void TestInitialiseWithPath( );
} ;
} // end of namespace
Something went wrong with that request. Please try again.