Incorrect grammar in `File` lock-related methods

[LeandroVandari]

Location

File::lock, File::try_lock, File::lock_shared, File::try_lock_shared

Summary

The recently stabilized lock-related methods for File have some grammatical errors.

1. All of the methods

If this file handle/descriptor, or a clone of it, already holds an lock [...]

Since lock starts with a consonant, the preceding article should be a.

Additionally, all of the methods refer to

acquire a [...] lock on the file

As I'm not a native english speaker, I'm not sure if this is correct, but it seems to me that

acquire a [...] lock to the file

is much more natural.

2. File::lock and File::try_lock

If the file not open for writing

should be

If the file is not open for writing

[LeandroVandari]

If this seems fine, I can open a PR for it later today.

[saethlin]

acquire a [...] lock on the file

This is the normal way to say what's going on, acquiring a lock to a file would be odd. At a guess this comes from the fact that we would say in the physical world that you put a lock on a door.

The other cases you've identified would be welcome fixes.

[LeandroVandari]

Apparently this is already fixed in master. I didn't see which commit changed it, but when cloning the repo it generates the docs with correct grammar.