typo in couple of places

Started by vignesh Cover 2 years ago6 messageshackers
Jump to latest
#1vignesh C
vignesh21@gmail.com

Hi,

I noticed a couple of typos in code. "the the" should have been "the",
attached patch has the changes for the same.

Regards,
Vignesh

Attachments:

0001-Fix-typo-in-couple-of-places.patchtext/x-patch; charset=US-ASCII; name=0001-Fix-typo-in-couple-of-places.patchDownload+2-3
In reply to: vignesh C (#1)
Re: typo in couple of places

vignesh C <vignesh21@gmail.com> writes:

Hi,

I noticed a couple of typos in code. "the the" should have been "the",
attached patch has the changes for the same.

This made me curious about other duplicate word occurrences, and after a
few minutes of increasingly-elaborate regexing to exclude false
postives, I found a few more (plus a bonus a _missing_ "the"). See the
attached patch (which includes your originl one, for completeness).

Regards,
Vignesh

- ilmari

Attachments:

0001-Fix-duplicate-words-in-docs-and-code-comments.patchtext/x-diffDownload+16-13
#3Amit Kapila
amit.kapila16@gmail.com
In reply to: Dagfinn Ilmari Mannsåker (#2)
Re: typo in couple of places

On Fri, Oct 6, 2023 at 8:52 PM Dagfinn Ilmari Mannsåker
<ilmari@ilmari.org> wrote:

vignesh C <vignesh21@gmail.com> writes:

Hi,

I noticed a couple of typos in code. "the the" should have been "the",
attached patch has the changes for the same.

This made me curious about other duplicate word occurrences, and after a
few minutes of increasingly-elaborate regexing to exclude false
postives, I found a few more (plus a bonus a _missing_ "the"). See the
attached patch (which includes your originl one, for completeness).

LGTM.

--
With Regards,
Amit Kapila.

#4vignesh C
vignesh21@gmail.com
In reply to: Dagfinn Ilmari Mannsåker (#2)
Re: typo in couple of places

On Fri, 6 Oct 2023 at 20:50, Dagfinn Ilmari Mannsåker <ilmari@ilmari.org> wrote:

vignesh C <vignesh21@gmail.com> writes:

Hi,

I noticed a couple of typos in code. "the the" should have been "the",
attached patch has the changes for the same.

This made me curious about other duplicate word occurrences, and after a
few minutes of increasingly-elaborate regexing to exclude false
postives, I found a few more (plus a bonus a _missing_ "the"). See the
attached patch (which includes your originl one, for completeness).

Thanks, Looks good.

Regards,
Vignesh

#5Amit Kapila
amit.kapila16@gmail.com
In reply to: vignesh C (#4)
Re: typo in couple of places

On Sat, Oct 7, 2023 at 8:28 AM vignesh C <vignesh21@gmail.com> wrote:

On Fri, 6 Oct 2023 at 20:50, Dagfinn Ilmari Mannsåker <ilmari@ilmari.org> wrote:

vignesh C <vignesh21@gmail.com> writes:

Hi,

I noticed a couple of typos in code. "the the" should have been "the",
attached patch has the changes for the same.

This made me curious about other duplicate word occurrences, and after a
few minutes of increasingly-elaborate regexing to exclude false
postives, I found a few more (plus a bonus a _missing_ "the"). See the
attached patch (which includes your originl one, for completeness).

Thanks, Looks good.

Pushed.

--
With Regards,
Amit Kapila.

#6vignesh C
vignesh21@gmail.com
In reply to: Amit Kapila (#5)
Re: typo in couple of places

On Mon, 9 Oct 2023 at 16:17, Amit Kapila <amit.kapila16@gmail.com> wrote:

On Sat, Oct 7, 2023 at 8:28 AM vignesh C <vignesh21@gmail.com> wrote:

On Fri, 6 Oct 2023 at 20:50, Dagfinn Ilmari Mannsåker <ilmari@ilmari.org> wrote:

vignesh C <vignesh21@gmail.com> writes:

Hi,

I noticed a couple of typos in code. "the the" should have been "the",
attached patch has the changes for the same.

This made me curious about other duplicate word occurrences, and after a
few minutes of increasingly-elaborate regexing to exclude false
postives, I found a few more (plus a bonus a _missing_ "the"). See the
attached patch (which includes your originl one, for completeness).

Thanks, Looks good.

Pushed.

Thanks for pushing this.