Thread Rating:
  • 0 Vote(s) - 0 Average
  • 1
  • 2
  • 3
  • 4
  • 5
PATCH: src/main.cpp various modifications.
06-05-2011, 01:41 PM,
#1
PATCH: src/main.cpp various modifications.
I've started a walk through all the code in "execution" order. I've started in src/main.cpp to have a better understanding of the code. Just because of that I've decided to rewrite the first file.

It now has the GPL header (as GPL says that every source file should have it but I understand it is not a problem), the "release_mouse" function is first declared after includes and not between them (just for better reading...), I've deleted some comments with old code that seemed unuseful (review them please), Now coments folow some rules (// First capital letter and a dot at the end.) and log now uses the words "INFO" and "ERROR" always.

Hope you use it.

http://www.mediafire.com/file/1um9bgy064...rite.patch
Reply
06-05-2011, 02:05 PM,
#2
 
Looks good. Can you prepend the std namespace to std components (std:Confusedtring). Hope to see more patches.
Reply
06-06-2011, 08:55 AM,
#3
 
NaN Wrote:Looks good. Can you prepend the std namespace to std components (std:Confusedtring). Hope to see more patches.

Sorry I sometimes have problems with English as I'm Spanish. What do you mean?

Code:
std::string
#include <string>
or
Code:
#include <string>
std::string

And now that I remember "Vásquez" or "Vázques" is from South America, I'm from Spain, Europe and I'm "Antonio Vázquez". I don't know if it was a simple misstype but it anyway it doesn't matter Smile. Thanks for your support.
Reply
06-06-2011, 01:45 PM,
#4
 
Quote:What do you mean?
Just add std:: (namespace qualifier) to string and other std components.

Quote:it was a simple misstype
Pure sloppiness on my part, sorry.
Reply


Forum Jump:


Users browsing this thread: 1 Guest(s)