Hi Liam, On Sun, 3 Dec 2017, Liam Beguin wrote: > The transform_todo_ids function is a little hard to read. Lets try > to make it easier by using more of the strbuf API. Also, since we'll > soon be adding command abbreviations, let's rename the function so > it's name reflects that change. I am not really a fan of the new name, and would prefer the old one, but that's only a nit, not a reason to reject the patch. The rest of it makes the code reads a lot nicer than before. Thank you, Johannes
