Re: RFC v3 - handling format strings (prinf)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 01/11/18 18:11, Ben Dooks wrote:
I'm not reposting the parsing bit, unless people would like it
again, it seems to have already been well reviewed.

This is a new evaluate patch and the test suite to add some
validation to go with it. It requires the ptr_list_nth_entry()
from Luc which i've fudged into the series. I've put a patch in
to help clean the validation output as it generates a lot of
files.

I've improved the printf handling since the previous version,
it now runs a table of formats and has handling for positional
arguments, widths, precisions and other fun bits of the printf
formatting that I am becoming to think is a bit crazy.

The current issues are:

I've still not got the pointer evaluationm right, but it sort
of works enough to get it tested. This will need a bit of work
but does that need to be done for the initial merge?

Do we want to have -Wformat as default for the initial merge?
If not i'll add a patch to default it off, so we can revert it
once this is sorted

I forgot to mention, do you think some of the small fixes and the
parsing bit can go on to a branch for merging? It would be nice to
have these out of the way.

--
Ben Dooks				http://www.codethink.co.uk/
Senior Engineer				Codethink - Providing Genius

https://www.codethink.co.uk/privacy.html



[Index of Archives]     [Newbies FAQ]     [LKML]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Trinity Fuzzer Tool]

  Powered by Linux