On 12/15/21 13:54, Paolo Valerio wrote: > Hello Mike, > > Mike Pattrick <[email protected]> writes: > >> As part of some previous checkpatch work, we discovered that checkpatch >> isn't always reporting correct line numbers. As it turns out, Python's >> splitlines function considers several characters to be new lines which >> common text editors do not typically consider to be new lines. For >> example, form feed characters, which this code base uses to cluster >> functionality. >> >> Signed-off-by: Mike Pattrick <[email protected]> >> --- >> utilities/checkpatch.py | 6 +++++- >> 1 file changed, 5 insertions(+), 1 deletion(-) >> > > tested it against multiple files, all reporting the correct line number. > The patch LGTM, > > Acked-by: Paolo Valerio <[email protected]>
Thanks! Applied. Best regards, Ilya Maximets. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
