Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removing "allows to" - tweet by Terrence Parr #2380

Closed
HansOlsson opened this issue May 24, 2019 · 7 comments · Fixed by #2685
Closed

Removing "allows to" - tweet by Terrence Parr #2380

HansOlsson opened this issue May 24, 2019 · 7 comments · Fixed by #2685
Labels
English Issue with English spelling, grammar, and/or punctuation. P: low

Comments

@HansOlsson
Copy link
Collaborator

https://twitter.com/the_antlr_guy/status/1119333987275534336

Non-native English speakers: please immediately, and with extreme prejudice, delete "allows to ..." from your internal grammar model. Drives us crazy! Thank you.

There are at least 12 uses of that construct in the current document, and it seems we can in those cases replace "allows to" by "allows us to ".

@HansOlsson
Copy link
Collaborator Author

On second thought it seems better to rewrite the sentences, instead of introducing the previously unused "us".

@HansOlsson HansOlsson added the English Issue with English spelling, grammar, and/or punctuation. label Jun 5, 2019
@xogeny
Copy link

xogeny commented Oct 9, 2019

I don't think there is anything wrong with "allows us to" (sounds natural enough to me). Another alternative would be "makes it possible to". The use of the infinitive here can also be tweaked so instead of "allows us " substitute "permits/allows ". For example,

This feature also allows to define multi-rate systems

becomes

This feature also permits defining multi-rate systems

But honestly, if I were speaking I'd almost certainly say "allows us to".

@HansOlsson
Copy link
Collaborator Author

HansOlsson commented Oct 17, 2019

I don't think there is anything wrong with "allows us to" (sounds natural enough to me).

Just to be clear: currently it just says "allows to", and I assume you agree that "allows to" isn't natural?

@svorkoetter
Copy link
Collaborator

Or more formally, "allows one to".

@mtiller
Copy link
Collaborator

mtiller commented Oct 19, 2019

Just to clarify...

Yes, I understood the current specification says "allows to" and you (Hans) were suggesting "allows us to" which I just wanted to indicate was fine (i.e., your solution is fine). But I wanted to just add a few more alternatives so that we didn't necessarily replace it with the same phrase everywhere. Stefan's additional alternative "allows one to" also sounds fine to me.

@beutlich
Copy link
Member

beutlich commented Oct 20, 2019

I am not a big fan of "allows one to" since my teachers at University of Bristol strictly insisted on avoiding it.

My examples:

  1. allows to compute - wrong
  2. allows us to compute - well, ok
  3. allows the user to compute - well, ok
  4. allows the computation - ok
  5. allows one to compute - should be avoided

Have a look at modelica-3rdparty/Modelica_DeviceDrivers@9fce550 where I fixed it for the conference article of the Modelica_DeviceDrivers library avoiding the Royal "one".

@svorkoetter
Copy link
Collaborator

svorkoetter commented Oct 20, 2019 via email

HansOlsson added a commit to HansOlsson/ModelicaSpecification that referenced this issue Oct 9, 2020
HansOlsson added a commit that referenced this issue Oct 12, 2020
* Avoid 'allows to'
Closes #2380
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
English Issue with English spelling, grammar, and/or punctuation. P: low
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants