LinuxCNC / linuxcnc

LinuxCNC controls CNC machines. It can drive milling machines, lathes, 3d printers, laser cutters, plasma cutters, robot arms, hexapods, and more.
http://linuxcnc.org/
GNU General Public License v2.0
1.75k stars 1.14k forks source link

Docs typos #2834

Closed smoe closed 6 months ago

smoe commented 6 months ago

As usual, something spotted while translating.

Please kindly confirm my interpretation "neither between" in docs/src/drivers/hal_gpio.adoc line 57. Admittedly this PR was mostly motivated as a technical excercise to work with WSL.

petterreinholdtsen commented 6 months ago

I assume you are talking about this sentence: The pin names are case-sensitive and there must be no spaces in the strings, neither between the comma-separated pins lists of the "=" signs.

I would write "no spaces in the string, nor between the comma-separated pins" instead myself. No native speaker, so not sure if it truly better.

hansu commented 6 months ago

Both is fine I guess. We can also write:

(3) The pin names are case-sensitive and there must be no spaces in the strings, not even between the comma-separated pins lists of the "=" signs.

or

(4) The pin names are case-sensitive and there must be neither spaces in the strings, nor between the comma-separated pins lists of the "=" signs.

But I would prefer (1) (Steffens version) or (2) (Petters version).

smoe commented 6 months ago

Heya, please then ("either" of you, pun intended) merge my request since this saves time, is apparently good enough, and closest to the original.

 

But I would prefer (1) (Steffens version) or (2) (Petters version).

hansu commented 6 months ago

I agree :)