maria-developers team mailing list archive

[Sergei Golubchik <serg@xxxxxxxxxxx>]

Hi, Alexey!

It's important for USER_TABLE to be last. And there can be only one last
value in the enum, if I add TABLES_MAX, someone will eventually put new
table just before TABLES_MAX (tests will catch it, but it might take
hours to track down to the actual reason).

So, it's safer to have just one last element in the enum, USER_TABLE.

On Dec 09, Alexey Botchkov wrote:
> Hello, Sergei!
> 
> I haven't found anything serious to improve there in your changes.
> sql_acl.cc gets nicer with this patch. So ok to push.
> 
> Only question left -
> -  TABLES_MAX // <== always the last
> +  USER_TABLE // <== always the last
> 
> Why not preserve the TABLES_MAX? Seems more self-explaining.
> +static LEX_CSTRING MYSQL_TABLE_NAME[USER_TABLE+1]=
> could be
> +static LEX_CSTRING MYSQL_TABLE_NAME[TABLES_MAX]=
> and a couple of similar places.
> 
> Best regards.
> HF

Regards,
Sergei
Chief Architect MariaDB
and security@xxxxxxxxxxx