Hi Phillip On 2019-06-13 17:45 UTC Phillip Wood <phillip.wood123@xxxxxxxxx> wrote: > >> + break; >> + default: >> + BUG(_("the control must not reach here")); > > This does not need to be translated as BUG() messages are not really for > users. Everything else looks fine to be now I know this is not intended for the users, but seeing a message like this a user might think to report it to the team with the steps for reproducing the BUG. Yes, I agree this needs a little bit of re-wording. > Best Wishes > > Phillip Thanks Rohit