Mailinglist Archive: opensuse-buildservice (312 mails)

< Previous Next >
[opensuse-buildservice] diff brp-checks 11.1 / factory
  • From: "Dominique Leuenberger" <Dominique.Leuenberger@xxxxxxxxxxxxx>
  • Date: Thu, 23 Jul 2009 10:02:46 +0200
  • Message-id: <4A6835460200002900026425@xxxxxxxxxxxxxxxxxxx>
Hi,

I was wondering why brp-checks are sometimes less strict than they are on
Factory.
Look at this short excerpt from a buildlog:

src/skin_main.cpp: In function 'void* Run(void*)':
src/skin_main.cpp:347: warning: control reaches end of non-void function
src/skin_main.cpp: At global scope:
src/skin_main.cpp:457: warning: 'int onSystrayChange(vlc_object_t*, const
char*, vlc_value_t, vlc_value_t, void*)' defined but not
used
src/skin_main.cpp:492: warning: 'int onTaskBarChange(vlc_object_t*, const
char*, vlc_value_t, vlc_value_t, void*)' defined but not
used

Ok, we all agree what the compiler warns about here. In 11.1, this is raised to
an error:
I: Program returns random data in a function
E: $pkg no-return-in-nonvoid-function src/skin_main.cpp:347

In 11.2 this passes cleanly the brp check (even though the compiler still warns
on it).

Now what would be the proper behavior? what 11.2 (aka Factory at the moment)
does or what 11.1 does?
Having a 'reaches end of non-void' on a function declared as void* looks weird
anyhow :) but that's another story.

If the 11.2 behavior is correct: how can I get 11.1 ignore this specific error?
(I don't want to completely disable brp-checks. I got to love them).
If the 11.1 behavior is correct: why did this vanish from the brp on Factory?

Best regards,
Dominique
--
To unsubscribe, e-mail: opensuse-buildservice+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: opensuse-buildservice+help@xxxxxxxxxxxx

< Previous Next >
Follow Ups