qgit icon indicating copy to clipboard operation
qgit copied to clipboard

Reduce compilation warnings

Open martenjj opened this issue 3 years ago • 1 comments

They do not affect the code, but give lots of diagnostic messages when compiled with -W -Wall -Wextra. The changes remove them so that other messages warning of potential problems will not be lost among the noise.

The wanings eliminated are: "declaration shadows a member" "use of old-style cast" "can be marked override" "declaration shadows a parameter" "declaration shadows a previous local" "implictly-declared (constructor) is deprecated"

martenjj avatar Nov 10 '22 10:11 martenjj

Hello Jonathan

For my instruction, please describe in some detail your compilation setup.

I have the latest gcc (gcc-12) available for my distro, I have the compiler debug flags set to -g -W -Wall -Wextra -pedantic and I don't see all the warnings you seem to see.

I can only see the warnings you fix with your last commit.

I can understand from the code the fixes in the 2nd and 3rd commit (although I would like to also get the warnings locally, so please help me with the config).

And I don't see any reasons in the code for the 1st and 4th commit.

Thanks in advance Cristian

tibirna avatar Dec 03 '22 14:12 tibirna