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 > <[email protected]> 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 <[email protected]> SUSE _______________________________________________ systemd-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/systemd-devel
