Konstantin Ryabitsev wrote: > I think it's reasonable to normalize \s+ into a single space for all queries, > right?
Yes, 3/3 does it for HTTP(S). Don't think it's needed for local queries. I'm leaving non-ASCII whitespace alone for now since I don't think it'd be a real problem. > Another observation is that when I missed a \ on one of the lines, I managed > to make lei unusable. :) That's a far more important problem fixed in 1/3 :> Eric Wong (3): lei: fix handling of broken lei.saved-search config files lei: pass client stderr to git-config in more places lei: normalize whitespace in remote queries lib/PublicInbox/Config.pm | 10 ++-- lib/PublicInbox/LEI.pm | 23 +++++++-- lib/PublicInbox/LeiEditSearch.pm | 79 +++++++++++++++++++++++++++++-- lib/PublicInbox/LeiMirror.pm | 4 +- lib/PublicInbox/LeiSavedSearch.pm | 77 ++++++------------------------ lib/PublicInbox/LeiXSearch.pm | 4 +- script/lei | 2 + t/lei-q-save.t | 17 +++++++ 8 files changed, 136 insertions(+), 80 deletions(-) -- unsubscribe: one-click, see List-Unsubscribe header archive: https://public-inbox.org/meta/
