Re: [PATCH 2/7] builtin: use a table for the builtins

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

 



On Fri, Jun 12, 2020 at 01:56:20AM +0100, Ramsay Jones wrote:
> On 10/06/2020 21:27, Luc Van Oostenryck wrote:
> > 
> > A table is table is preferable but a complication for doing this
> 
> s/table is table is/table is/

Thanks for noticing this typo (and the ones in the other patches).
 
> > +static void declare_one_builtin(const struct builtin_fn *entry)
> > +{
> > +	struct symbol *sym = create_symbol(0, entry->name, SYM_NODE, NS_SYMBOL);
> 
> So, assuming stream 0 here as well ...
> 
> > -static void declare_builtin(const char *name, struct symbol *rtype, int variadic, ...)
> > -{
> > -	int stream = 0;			// FIXME
> 
> ... and, yes, here is the FIXME (so my memory is not as bad
> as I thought!).
> 
> This all looks good.

Well yes ... I don't really like the situation, though.
I probably should do something like adding a #define builtin_stream 0
and use this.
 
Thnaks to bring my attention to this.
-- Luc



[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