Jorgen,
Thanks for the comments.
> STATUS:
> -------
> Approved, but please reply to the question below
>
> REQUIREMENTS:
> -------------
> None
>
> OPTIONAL:
> ---------
> 1) Modify slightly wrong comment in test
Ok, will fix.
> 2) Add test for BACKUP DATABASE * for user with rights.
Ok, will add one.
> QUESTION:
> ---------
> Q1) What is EXTRA_ACL?
This was an unused ACL define that the architects agreed I could remove so that
I would not have to extend the size of the define to a ulonglong. :) As it was,
the original code had only 1 position left and I needed 2 (BACKUP_ACL,
RESTORE_ACL). Good question!
> COMMENTARY:
> -----------
> Thank you for all the hard work in satisfying the reviewers. I know it's
> been a bumpy ride on this bug...
YW. Yes, it has.
Chuck
