[systemd-devel] PATCH: fix sparse warnings

[Frederic Crozat]

Le jeudi 22 mars 2012 à 10:32 -0700, Kok, Auke-jan H a écrit :
> On Wed, Mar 21, 2012 at 5:21 PM, Lennart Poettering
> <lennart at poettering.net> wrote:
> >> Here is a patch integrating your header (modified as wanted by Lennart)
> >> and the changes in various locations of journal to use le64_t.
> >>
> >> It also fixes some potential endianness errors, please review it
> >> carefully.
> >
> > Looks good to me. Applied!
> >
> > Now, one more question, since I actually never used sparse myself:
> > what's the best way to make use of this? If I run autogen and build this
> > with CC=cgcc some things look really borked (like generation of the dbus
> > introspection files goes to stdout). Any suggestions?
> 
> I'd suggest including this in some 'make test' type target only, but
> note that most of the stuff sparse throws our are warnings, and so any
> form of automation isn't really helpful with sparse.

agreed. And be sure to run a "git" sparse, I'm sent some fixes regarding
some barrier code which is used by systemd.

> Sparse checking is great, just like many other tools like valgrind,
> etc. , but, skilled eyes are required to make sense of the output.

Yep.


-- 
Frederic Crozat <fcrozat at suse.com>
SUSE