Mike Frysinger <[email protected]> writes:

> On Monday 21 May 2012 19:05:59 Mike Frysinger wrote:
>> The current cleanup_html_url code warns because it's marked static
>> but no one calls it.  Reading the code base, we see it only gets
>> called when DEBUG_MALLOC is defined, so change the static markings
>> to check that ifdef.

thanks for the patch, but I think these functions should not be defined
as static.  It would be fine to guard static functions used in the same
compilation unit but I prefer to define functions used from different
files in any case.

Cheers,
Giuseppe


Reply via email to