As the patch weights 2.3 Mb I'm attaching it here.
Created attachment 39384 [details]
Remove tons of comments
Heh - sooo ... I'm torn on this one. Large automated code-cleanups like this are ok I guess but can create some merge problems.
Also - while I dislike the // ------------- stuff - it is nice to have at least a blank-line separating functions to make them more readable, and there is a good amount of:
stuff in there :-) any chance of a new line instead ? or was this done manually ?
Oh ! and I forgot to say - thanks for your patch ! :-) it's wonderful to have you contributing, sorry to be slightly nervous this once.
Created attachment 39397 [details]
Remove comments, death code and trailing spaces
Sorry to send such a big patch before, here is a small more well-formed one.
It "fixes" the code for base/dbaccess/source/core/api/* files.
Created attachment 39398 [details]
Removed comments, trailing spaces and death code (2)
This patch cleans the base/dbaccess/source/core/dataaccess/*.* files
Created attachment 39400 [details]
Removed comments, trailing spaces and death code (3)
This patch cleans the base/dbaccess/source/core/inc/*.* files
Thanks for the patches!
I pushed them to the base repo:
Closing - Sophie