On Sun, May 2, 2021 at 4:27 AM Alvaro Herrera <alvhe...@alvh.no-ip.org> wrote: > > On 2021-May-01, Bharath Rupireddy wrote: > > > IMO, it's not good to change the function API just for showing up > > parse_position (which is there for cosmetic reasons I feel) in an error > > which actually has the option name clearly mentioned in the error message. > > Why not? We've done it before, I'm sure you can find examples in the > git log. The function API is not that critical -- these functions are > mostly only called from ProcessUtility and friends.
I feel it is better to include it wherever possible. Regards, Vignesh
