public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* 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).