FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > ArchLinux > ArchLinux Pacman Development

 
 
LinkBack Thread Tools
 
Old 04-03-2011, 07:35 AM
Xavier Chantry
 
Default Remove dead stores reported by clang-analyzer

On Sun, Apr 3, 2011 at 8:14 AM, Sebastian Nowicki <sebnow@gmail.com> wrote:
> Signed-off-by: Sebastian Nowicki <sebnow@gmail.com>
> ---
> *lib/libalpm/be_local.c | * *2 +-
> *lib/libalpm/pkghash.c *| * *3 +--
> *2 files changed, 2 insertions(+), 3 deletions(-)
>

I didn't send these yesterday to the ML as they looked trivial enough,
but maybe I should have.
http://code.toofishes.net/cgit/xavier/pacman.git/log/?h=working
 
Old 04-03-2011, 08:15 AM
Sebastian Nowicki
 
Default Remove dead stores reported by clang-analyzer

On Sun, Apr 3, 2011 at 3:35 PM, Xavier Chantry <chantry.xavier@gmail.com>wrote:

> On Sun, Apr 3, 2011 at 8:14 AM, Sebastian Nowicki <sebnow@gmail.com>
> wrote:
> > Signed-off-by: Sebastian Nowicki <sebnow@gmail.com>
> > ---
> > lib/libalpm/be_local.c | 2 +-
> > lib/libalpm/pkghash.c | 3 +--
> > 2 files changed, 2 insertions(+), 3 deletions(-)
> >
>
> I didn't send these yesterday to the ML as they looked trivial enough,
> but maybe I should have.
> http://code.toofishes.net/cgit/xavier/pacman.git/log/?h=working
>
>
I thought you might have already done this when you found the open_mode bug
from clang-analyzer .
 
Old 04-05-2011, 02:07 AM
Allan McRae
 
Default Remove dead stores reported by clang-analyzer

On 04/04/11 18:58, Xavier wrote:

Allan McRae wrote:


Please send patches to the list so that we can complain about issues like:

aba191d5 (Xavier Chantry 2011-04-03 10:28:09 +0200 276)
printf("%lc", *p);

util.c: In function ‘indentprint’:
util.c:276:3: error: format ‘%lc’ expects argument of type ‘wint_t’,
but argument 2 has type ‘wchar_t’ [-Werror=format]
cc1: all warnings being treated as errors




This one was not supposed to be committed, I know Dan would have rejected it
anyway.
I was just pissed off by this dumb issue, and I did not manage to make gcc
break. Is that with gcc 4.6 (I was on 4.5) ?



OK.

That was with a gcc-4.7 snapshot (which is still quite close to what
gcc-4.6 is).


Allan
 

Thread Tools




All times are GMT. The time now is 03:13 PM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org