From: Petr Vorel <pvorel@suse.cz>
To: ltp@lists.linux.it
Subject: [LTP] [PATCH 1/2] docparse: Replace \t with space
Date: Thu, 21 Jan 2021 16:43:55 +0100 [thread overview]
Message-ID: <YAmhO8gMJ1KZ4EHA@pevik> (raw)
In-Reply-To: <YAmUwVWvm84w8ITB@yuki.lan>
Hi,
> > JQ says "control characters from U+0000 through U+001F must be
> > escaped". So I expect it is only a matter of time until some other
> > control character is used.
+1
> > Perhaps we should escape all control characters into the \uXXXX
> > hexidecimal form?
> Or fail the compilation if we get one of these into the parser?
We do fail already, but it's a bit hidden now.
I don't know why build continues for so long.
Also there would have to be striking error message.
But docparser is not mandatory (people might have it disabled), thus mostly it
will be us the maintainers who is going to fix these whitespace issues :(.
> There is no point in having them in the metadata anyways.
That would be solved by replacing some reasonable subset. So far it was the tab
character.
Kind regards,
Petr
next prev parent reply other threads:[~2021-01-21 15:43 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-01-21 13:00 [LTP] [PATCH 1/2] docparse: Replace \t with space Petr Vorel
2021-01-21 13:00 ` [LTP] [PATCH 2/2] testinfo.pl: Optimize imports Petr Vorel
2021-01-28 14:53 ` Petr Vorel
2021-01-21 13:05 ` [LTP] [PATCH 1/2] docparse: Replace \t with space Petr Vorel
2021-01-21 14:47 ` Richard Palethorpe
2021-01-21 14:50 ` Cyril Hrubis
2021-01-21 15:43 ` Petr Vorel [this message]
2021-01-21 15:59 ` Richard Palethorpe
2021-01-21 16:20 ` Petr Vorel
2021-01-21 16:28 ` Cyril Hrubis
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=YAmhO8gMJ1KZ4EHA@pevik \
--to=pvorel@suse.cz \
--cc=ltp@lists.linux.it \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.