[U-Boot] what is the rationale for 900+ lines duplicated in common/dlmalloc.c and include/malloc.h?

Robert P. J. Day rpjday at crashcourse.ca
Thu Apr 21 13:21:54 CEST 2016


On Wed, 20 Apr 2016, Tom Rini wrote:

> On Sat, Apr 16, 2016 at 04:23:27AM -0400, Robert P. J. Day wrote:
>
> >   just noticed, in common/dlmalloc.c, the sizable chunk:
> >
> >     #if 0   /* Moved to malloc.h */
> >     /* ---------- To make a malloc.h, start cutting here ------------ */
> >
> >     ... several hundred lines ...
> >
> >     #ifdef __cplusplus
> >     };  /* end of extern "C" */
> >     #endif
> >
> >     /* ---------- To make a malloc.h, end cutting here ------------ */
> >     #endif  /* 0 */                 /* Moved to malloc.h */
> >
> > given that those lines appear to be duplicated in malloc.h, what is
> > the purpose of them still being in dlmalloc.c? i'm just curious.
>
> Good point.  Something that could probably be cleaned up.

  i can submit a patch for that ... i can either remove the "if 0"
stuff in its entirety, or leave a short comment to the effect that
what was here has now been moved, if you think it's worth it. pick
one, and i'll put together the patch.

rday

-- 

========================================================================
Robert P. J. Day                                 Ottawa, Ontario, CANADA
                        http://crashcourse.ca

Twitter:                                       http://twitter.com/rpjday
LinkedIn:                               http://ca.linkedin.com/in/rpjday
========================================================================



More information about the U-Boot mailing list