[tarantool-patches] Re: [PATCH v2 1/5] sql: introduce structs assembling DDL arguments during parsing

Konstantin Osipov kostja at tarantool.org
Thu Jan 24 15:30:59 MSK 2019


* n.pettik <korablev at tarantool.org> [19/01/24 13:51]:
> >> +	 * One of parse_def structures which are used to
> >> +	 * assemble and carry arguments of DDL routines
> >> +	 * from parse.y
> >> +	 */
> >> +	void *alter_entity_def;
> > 
> > Please consider adding a type code to the base entity def, so that
> > you can use a base struct reference rather than void *. 
> 
> Please, look at code and explain why do we need type
> code? At first I added it, but it turned out that such code
> was completely useless. The only real usage was to
> assert that type of structure matches with its code.

To assert the structure matches with its code.

-- 
Konstantin Osipov, Moscow, Russia, +7 903 626 22 32
http://tarantool.io - www.twitter.com/kostja_osipov




More information about the Tarantool-patches mailing list