Hello, I made the modification in the commit message and opened a PR.
I both modified a configuration file and sent the character encoding as a parameter when I sent the patch. I'll have to see what went wrong with the mailer. Thank you, Adela În sâm., 7 nov. 2020 la 08:45, Akim Demaille <[email protected]> a scris: > Hi Adela, > > I can't install this patch, it was mangled by your mailer. Please send > it as an attachment, or also provide a PR. > > > Le 6 nov. 2020 à 16:52, Adela Vais <[email protected]> a écrit : > > > > Fixed whitespace issues and removed the @property attribute. > > * examples/d/calc/calc.y, examples/d/simple/calc.y: Here. > > Make this: > > * examples/d/calc/calc.y, examples/d/simple/calc.y: Fixe whitespace issues > and remove the @property attribute. > > > I use "here" when the commit title is clear enough, but if you > have to spell something different, do it with the file name, > and imperative. > > > @@ -72,12 +72,12 @@ import std.range.primitives; > > import std.stdio; > > > > auto calcLexer(R)(R range) > > - if (isInputRange!R && is (ElementType!R : dchar)) > > +if (isInputRange!R && is(ElementType!R : dchar)) > > { > > return new CalcLexer!R(range); > > } > > I'm surprised by this. That style came from H.S., and clearly > shows that that 'if' is compile-time, and controls the > availability of the function. > > But you are right, the D coding style does not use indentation > here (https://dlang.org/dstyle.html#phobos_declarations). > I preferred H.S.'s style, but you're change is right. > > > Cheers!
