svn commit: r318913 - in head/devel/libcfu: . files

[Alexey Dokuchaev]

On Fri, May 24, 2013 at 07:08:56AM +0100, Chris Rees wrote:
> On 24 May 2013 04:37, "Alexey Dokuchaev" <danfe at freebsd.org> wrote:
> > On Thu, May 23, 2013 at 10:05:22PM +0000, Chris Rees wrote:
> > > New Revision: 318913
> > > URL: http://svnweb.freebsd.org/changeset/ports/318913
> > >
> > > Log:
> > >   New versions of clang complain about assigning variables to
> > >   themselves.
> > >
> > >   Let's just remove -Werror and ignore the warnings.
> >
> > Hmm, wouldn't it be better to fix these bugs (judging from commit log it
> > should not be too hard) and send the patch upstream?
> 
> They are just example files, doing silly things such as;
> 
> argc = argc;
> 
> Trouble is, removing these leads to "unused variable" warnings, so really
> it's better to ignore the warnings.

OK, I see.  But wouldn't casting to void help?

	(void)argc;

It could be even automated with simple sed(1) expression (REINPLACE_CMD) to
avoid big cumbersome patch...

./danfe