* Substitute all "the the" with "the"
@ 2019-01-10 7:44 Дилян Палаузов
2019-01-10 17:23 ` Joseph Myers
0 siblings, 1 reply; 2+ messages in thread
From: Дилян Палаузов @ 2019-01-10 7:44 UTC (permalink / raw)
To: gcc-patches
⦠by applying
sed -i "s/the the/the/" `git grep -l "the the"`
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: Substitute all "the the" with "the"
2019-01-10 7:44 Substitute all "the the" with "the" Дилян Палаузов
@ 2019-01-10 17:23 ` Joseph Myers
0 siblings, 0 replies; 2+ messages in thread
From: Joseph Myers @ 2019-01-10 17:23 UTC (permalink / raw)
To: Дилян
Палаузов
Cc: gcc-patches
[-- Attachment #1: Type: text/plain, Size: 738 bytes --]
On Thu, 10 Jan 2019, ÐилÑн ÐалаÑзов wrote:
> sed -i "s/the the/the/" `git grep -l "the the"`
That looks wrong; there are plenty of instances of "the theory", "the then
branch" and similar that should not have such a substitution applied.
See Sandra's patch submissions for "can not", and the discussion thereof,
for examples of how to propose such changes, including various files and
directories in the source tree that should be excluded because they e.g.
come from upstream sources maintained through different processes, or in
the case of testcases because such fixes are not particularly relevant to
them and unnecessarily perturb the code.
--
Joseph S. Myers
joseph@codesourcery.com
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-01-10 17:23 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-10 7:44 Substitute all "the the" with "the" Дилян Палаузов
2019-01-10 17:23 ` Joseph Myers
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).