Re: [patch] Remove binc from vi(1)

2017-07-24 Thread Ingo Schwarze
Hi Martijn, On 06/22/17 21:32, Martijn van Duren wrote: > Attached a patch to remove the binc function from vi > and replace it with recallocarray. In principle, the memory handling in vi is very ugly, and getting it into a more standard form seems desirable - but likely to cause quite some

Re: [patch] Remove binc from vi(1)

2017-07-13 Thread Martijn van Duren
No one? On 07/02/17 19:58, Martijn van Duren wrote: > Any takers? > > On 06/22/17 21:32, Martijn van Duren wrote: >> Hello tech@, >> >> Attached a patch to remove the binc function from vi and replace it with >> recallocarray. The functions effectively do the same thing since >> BINC_{GOTO,RET}

Re: [patch] Remove binc from vi(1)

2017-07-02 Thread Martijn van Duren
Any takers? On 06/22/17 21:32, Martijn van Duren wrote: > Hello tech@, > > Attached a patch to remove the binc function from vi and replace it with > recallocarray. The functions effectively do the same thing since > BINC_{GOTO,RET} already do the nlen > llen comparison. I've run > this without

[patch] Remove binc from vi(1)

2017-06-22 Thread Martijn van Duren
Hello tech@, Attached a patch to remove the binc function from vi and replace it with recallocarray. The functions effectively do the same thing since BINC_{GOTO,RET} already do the nlen > llen comparison. I've run this without any issues, but since recallocarray does extra checks and binc ALWAYS