bug#31800: suggestion of improvement for sort-numeric-fields function.

[Stefan Kangas]

Lars Ingebrigtsen <larsi@gnus.org> writes:

>>> -                (sort-skip-fields field)
>>> +                     (unless (string-match-p "^\\s-*$" (thing-at-point 
>>> 'line))
>>> +                       (sort-skip-fields field))
>
> I don't think this patch makes sense as is -- I think that's a
> convoluted way of saying `looking-at'?  But it makes conceptual sense, I
> think.

I'm not sure.  Do you mean to use:

    (looking-at "\\s*$")

Are we sure point is at the beginning of line here though?  The above
code doesn't require that, but also seems to be wrong if region is in
the middle of a line.  But I'm not sure how people typically use this
in that case.

BTW, shouldn't we also fix the same bug in sort-fields while we're at
it?

Best regards,
Stefan Kangas