On Fri, Feb 22, 2019 at 06:50:06PM +0300, Olga Telezhnaya wrote: > In my opinion, it still has some issues. I mentioned all of them in > TODOs in comments. All of them considered to be separate tasks for > other patches. Some of them sound easy and could be great tasks for > newbies. One other thing I forgot to mention: your patches ended up on the list in jumbled order. How do you send them? Usually `send-email` would add 1 second to the timestamp of each, so that threading mail readers sort them as you'd expect (even if they arrive out of order due to the vagaries of SMTP servers). -Peff
