[Geany-Devel] Plugin-Maintainer: Please review PR and comment on them/merge them

Pavel Roschin roshin at xxxxx
Fri Mar 28 17:31:57 UTC 2014


> 
> 
> Am 28.03.2014 10:00, schrieb Lex Trotman:
> > On 28 March 2014 19:46, Frank Lanitz <frank at frank.uvena.de> wrote:
> >> Am 28.03.2014 01:10, schrieb Lex Trotman:
> >>> The one you made on Geanypy has already been improved by upstream
> >>> (Matthew) so you should just commit that improvement.
> >>
> >> I was planning to have a look onto it Sunday (you mean #116, don't
> >> you?). But this is not a feature, only kind of a bug fix ;)
> > 
> > Yep that one, I guess its more a feature since there is no bug (in
> > Geanypy), more its a bug in cppcheck :)
> 
> 
> ... well.... puh... At least it's not changing any string ;)
> However, will do it at the weekend.
> 
> cheers,
> Frank
> _______________________________________________
> Devel mailing list
> Devel at lists.geany.org
> https://lists.geany.org/cgi-bin/mailman/listinfo/devel


Here is definitely false-positeve. cppcheck was confused by a terrible define:
return operator inside... Without return operator of course here could be a bug.

--
Best regards,
Pavel Roschin aka RPG


More information about the Devel mailing list