Re: [PATCH 1/8] malloc: implement USE_DL_PREFIX via inline functions

2020-03-04 Thread Simon Glass
Hi Simon, On Wed, 4 Mar 2020 at 13:45, Simon Goldschmidt wrote: > > Am 20.02.2020 um 04:05 schrieb Simon Glass: > > On Wed, 19 Feb 2020 at 12:39, Simon Goldschmidt > > wrote: > >> > >> Commit cfda60f99ae2 ("sandbox: Use a prefix for all allocation functions") > >> introduced preprocessor macros

Re: [PATCH 1/8] malloc: implement USE_DL_PREFIX via inline functions

2020-03-04 Thread Tom Rini
On Wed, Mar 04, 2020 at 09:44:27PM +0100, Simon Goldschmidt wrote: > Am 20.02.2020 um 04:05 schrieb Simon Glass: > > On Wed, 19 Feb 2020 at 12:39, Simon Goldschmidt > > wrote: > >> > >> Commit cfda60f99ae2 ("sandbox: Use a prefix for all allocation functions") > >> introduced preprocessor macros

Re: [PATCH 1/8] malloc: implement USE_DL_PREFIX via inline functions

2020-03-04 Thread Simon Goldschmidt
Am 20.02.2020 um 04:05 schrieb Simon Glass: > On Wed, 19 Feb 2020 at 12:39, Simon Goldschmidt > wrote: >> >> Commit cfda60f99ae2 ("sandbox: Use a prefix for all allocation functions") >> introduced preprocessor macros for malloc/free etc. >> >> This is bad practice as it essentially makes 'free'

Re: [PATCH 1/8] malloc: implement USE_DL_PREFIX via inline functions

2020-02-19 Thread Simon Glass
On Wed, 19 Feb 2020 at 12:39, Simon Goldschmidt wrote: > > Commit cfda60f99ae2 ("sandbox: Use a prefix for all allocation functions") > introduced preprocessor macros for malloc/free etc. > > This is bad practice as it essentially makes 'free' a reserved keyword and > resulted in quite a bit of

[PATCH 1/8] malloc: implement USE_DL_PREFIX via inline functions

2020-02-19 Thread Simon Goldschmidt
Commit cfda60f99ae2 ("sandbox: Use a prefix for all allocation functions") introduced preprocessor macros for malloc/free etc. This is bad practice as it essentially makes 'free' a reserved keyword and resulted in quite a bit of renaming to avoid that reserved keyword. A better solution is to