On Tue, Sep 15, 2015 at 8:37 PM, Ted Unangst wrote:
> Miod Vallat wrote:
> > > Hi,
> > >
> > > Here a first sets of "if(x) free(x)" cleanup in sys/arch/
> > >
> > > This patch contains only trivial if(x) removal. The size argument in
> > > free is keep untouched (because it is already setted, or
> > The arch/*/stand use a free() implementation which does not accept NULL
> > pointers (see sys/lib/libsa/alloc.c).
>
> But we're going to fix that too, right?
Yes. Developers those days are expecting free(NULL) to work, and as long
as it's easy to achieve, there is no reason to deprive them fr
Miod Vallat wrote:
> > Hi,
> >
> > Here a first sets of "if(x) free(x)" cleanup in sys/arch/
> >
> > This patch contains only trivial if(x) removal. The size argument in
> > free is keep untouched (because it is already setted, or because it
> > makes sens to keep it to 0).
> >
> > Comments ? OK
Miod Vallat wrote:
> > Hi,
> >
> > Here a first sets of "if(x) free(x)" cleanup in sys/arch/
> >
> > This patch contains only trivial if(x) removal. The size argument in
> > free is keep untouched (because it is already setted, or because it
> > makes sens to keep it to 0).
> >
> > Comments ? OK
> Hi,
>
> Here a first sets of "if(x) free(x)" cleanup in sys/arch/
>
> This patch contains only trivial if(x) removal. The size argument in
> free is keep untouched (because it is already setted, or because it
> makes sens to keep it to 0).
>
> Comments ? OK ?
The arch/*/stand use a free() imp
Hi,
Here a first sets of "if(x) free(x)" cleanup in sys/arch/
This patch contains only trivial if(x) removal. The size argument in
free is keep untouched (because it is already setted, or because it
makes sens to keep it to 0).
Comments ? OK ?
--
Sebastien Marie
Index: b/sys/arch/amd64/stand/l
