Re: [PATCH v3 0/1] userdiff: improve Bash function and word regex patterns

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Moumita <dhar61595@xxxxxxxxx> writes:

>     + ## t/t4018/bash-bashism-style-multiline-function (new) ##
>     +@@
>     ++function RIGHT \
>     ++{    
>     ++    echo 'ChangeMe'
>     ++}
>     + \ No newline at end of file
>     +
>     + ## t/t4018/bash-posix-style-multiline-function (new) ##
>     +@@
>     ++RIGHT() \
>     ++{
>     ++    ChangeMe
>     ++}
>     + \ No newline at end of file

For these new test, is it essential that these sample files end in
incomplete lines?  In other words, are these tests trying to make
sure that the function line is correctly found even if the function
body is at the end of the file that lack the final terminating LF?

If that is what they are testing, please add comments near the
beginning of the file to tell future developers that it is essential
that they keep these files end in incomplete lines and why.

If that is not what these tests are checking, then make these lines
complete lines instead, as they waste future developers' time making
them wonder if there are valid reasons why these files must end in
incomplete lines.





[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux