[pve-devel] applied: [PATCH common 1/2] Fix 2339: Handle multiple blank lines correctly

Thomas Lamprecht t.lamprecht at proxmox.com
Wed Sep 4 16:22:57 CEST 2019


On 28.08.19 11:22, Fabian Ebner wrote:
> It turns out that the line number counting was also broken (even on files without multiple blanks), since the body of the while inside the nextline subroutine would not be executed for a blank.
> I guess the subroutine was intended to skip comments and blanks, but since we use blanks to recognize the end of a section, I changed it to only skip comments.
> 
> Signed-off-by: Fabian Ebner <f.ebner at proxmox.com>
> ---
> Changes from v1:
> * fix line number counting
> * style
> 

applied, much thanks for this! Quite the old bug, great that you run
into it and fixed both!




More information about the pve-devel mailing list