Merge lp:~dcow90/myro-c++/extern-c into lp:myro-c++

[David Cowden]

The change is described in detail in the commit. I changed one function prototype so it is compatible with the use of extern "C" (I believe that was the issue). My guess is, when using extern "C", functions overloaded with a void version and a null string version:

function();

and

function (string s = "");

break down both into the same signature to the compiler. Which is why the compiler was complaining.

Simply changing the prototype to:

function (string s);

fixes the error and allows us to compile against the Scribbler header.

Please feel free to ask any questions. This was a confusing error to come across and one that didn't manifest until recently and under special conditions so I may need to explain it better.

[John Hoare (jhoare)]

review: Approve

=== modified file 'headers/Scribbler.h'
--- headers/Scribbler.h	2011-06-28 01:17:27 +0000
+++ headers/Scribbler.h	2011-07-01 21:08:32 +0000
@@ -433,7 +433,7 @@
 	 * @return An integer represent the light intensity for the specified
 	 * window.
 	 */
-	int getBright(std::string value = ""); 
+	int getBright(std::string value); 
 
 	/**
 	 * Used to query a specified light sensor located on the rear of the