On 08.02.25 02:56, Jacob Champion wrote:
+ oauth_json_set_error(ctx, /* don't bother translating */
With the project style format for translator comments this should be:
+ /* translator: xxx */
+ oauth_json_set_error(ctx,
This comment was just meant to draw attention to the lack of
libpq_gettext(). Does it still need a translator note if we don't run
it through translation?
No, that wouldn't have any effect.
I think you can just remove that comment. It's pretty established that
internal errors don't need translation, so it would be understood from
looking at the code.
