comprehensive-rust icon indicating copy to clipboard operation
comprehensive-rust copied to clipboard

comparison: copyedit 'delays' to 'delayed'

Open riking opened this issue 3 years ago • 3 comments

This is the correct grammatical form for the bullet point, it expands into "Destructors [of objects are] delayed."

riking avatar Dec 27 '22 22:12 riking

Hi @riking,

This is the correct grammatical form for the bullet point, it expands into "Destructors [of objects are] delayed."

Thanks for catching this. As you've guessed, I'm not a native speaker so I appreciate these grammar fixes a lot!

I think I actually wanted to say "Destructor delays" now that I see it again. How do you find that form?

mgeisler avatar Dec 28 '22 08:12 mgeisler

Hm, I think that works - one focusing on the fact that work is batched together, the other focusing on the fact that each individual object is separately delayed.

On Wed, Dec 28, 2022, 12:28 AM Martin Geisler @.***> wrote:

Hi @riking https://github.com/riking,

This is the correct grammatical form for the bullet point, it expands into "Destructors [of objects are] delayed."

Thanks for catching this. As you've guessed, I'm not a native speaker so I appreciate these grammar fixes a lot!

I think I actually wanted to say "Destructor delays" now that I see it again. How do you find that form?

— Reply to this email directly, view it on GitHub https://github.com/google/comprehensive-rust/pull/79#issuecomment-1366463599, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEZJM237KNR72WQJ7IAQETWPP23ZANCNFSM6AAAAAATKYA23A . You are receiving this because you were mentioned.Message ID: @.***>

riking avatar Dec 28 '22 14:12 riking

I think that works - one focusing on the fact that work is batched together, the other focusing on the fact that each individual object is separately delayed.

Okay, great! Will you push the change to your branch or should I do it?

mgeisler avatar Dec 28 '22 14:12 mgeisler