From mboxrd@z Thu Jan 1 00:00:00 1970 From: Petr Vorel Date: Thu, 21 Jan 2021 16:43:55 +0100 Subject: [LTP] [PATCH 1/2] docparse: Replace \t with space In-Reply-To: References: <20210121130033.20764-1-pvorel@suse.cz> <87mtx29wiq.fsf@suse.de> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: ltp@lists.linux.it 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