On Thu, Mar 14, 2024 at 7:23 PM Daniel Gustafsson <dan...@yesql.se> wrote: > > > On 14 Mar 2024, at 14:21, Robert Treat <r...@xzilla.net> wrote: > > On Thu, Mar 14, 2024 at 8:21 AM Daniel Gustafsson <dan...@yesql.se> wrote: > > >> - canceling connection in <application>psql</application> wouldn't > >> cancel > >> + canceling a connection in <application>psql</application> will not > >> cancel > >> Nitpickery (perhaps motivated by english not being my first language), but > >> since psql only deals with one connection I would expect this to read "the > >> connection". > >> > > > > My interpretation of this is that "a connection" is more correct > > because it could be your connection or someone else's connection (ie, > > you are canceling one of many possible connections). Definitely > > nitpickery either way. > > Fair point. > > >> - * Returns true iff the lock was acquired. > >> + * Returns true if the lock was acquired. > >> Using "iff" here is being consistent with the rest of the file (and > >> technically > >> correct): > > > Ah, yeah, I was pretty focused on the event trigger stuff and didn't > > notice it being used elsewhere; thought it was a typo, but I guess > > it's meant as shorthand for "if and only if", I wonder how many people > > are familiar with that. > > I would like to think it's fairly widely understood among programmers, but I > might be dating myself in saying so. > > I went ahead and applied this with the fixes mentioned here with one more tiny > change to the last hunk of the patch to make it say "login event trigger" > rather than just "login trigger". Thanks for the submission! >
LGTM, thanks! Robert Treat https://xzilla.net