Re: [PATCH v2 -perfbook] howto, intro, cpu: Break and capitalize after colon

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jun 14, 2021 at 07:58:59AM +0900, Akira Yokosawa wrote:
> On Sun, 13 Jun 2021 08:57:25 -0700, Paul E. McKenney wrote:
> > On Sat, Jun 12, 2021 at 03:14:52PM +0900, Akira Yokosawa wrote:
> >> Fix violations detected by the check scripts at hand.
> >>
> >> Signed-off-by: Akira Yokosawa <akiyks@xxxxxxxxx>
> > 
> > I have pulled this in, thank you very much!
> > 
> > There is one mistake I made in my previous reply, and I have fixed
> > this.  Please see below.
> > 
> [...]
> >> diff --git a/cpu/hwfreelunch.tex b/cpu/hwfreelunch.tex
> >> index 92f04f16..68c6dd5f 100644
> >> --- a/cpu/hwfreelunch.tex
> >> +++ b/cpu/hwfreelunch.tex
> >> @@ -167,8 +167,9 @@ That said, they may be necessary steps on the path to the late Jim Gray's
> >>  \label{sec:cpu:Novel Materials and Processes}
> >>  
> >>  Stephen Hawking is said to have claimed that semiconductor manufacturers
> >> -have but two fundamental problems: (1) the finite speed of light and
> >> -(2) the atomic nature of matter~\cite{BryanGardiner2007}.
> >> +have but two fundamental problems:
> >> +(1)~The finite speed of light and
> >> +(2)~the atomic nature of matter~\cite{BryanGardiner2007}.
> > 
> > I should have capitalized the "the" in this last line.  I did so before
> > applying your patch.
> 
> Well, I'm afraid this pattern is well beyond the capability of the dumb
> perl script.  (Hint: It only looks at the next input line of colon.)

I cannot complain, given that you have taken the checking scripts far
beyond what I expected.  Having a few manual checks still required
is no dishonor.  ;-)

> Anyway, I'll update the other chapters in the same manner.

Thank you very much!

							Thanx, Paul



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux