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

Revert commit 60007ff #178

Closed
wants to merge 5 commits into from
Closed

Revert commit 60007ff #178

wants to merge 5 commits into from

Conversation

norwd
Copy link

@norwd norwd commented Dec 28, 2024

The rewording doesn't read well to me, so for consideration, this is my suggestion to resolve #176. Instead of changing the wording, this just changes the word "applauding" to "cheering" and that makes the inconsistency go away with much less changes to the prose.

rrthomas and others added 2 commits December 26, 2024 15:26
See #176. Originally Harry “only” applauded two people. Instead, guess that
when talking about Zabini, the point is that he meant to start applauding
Slytherins, but only managed to applaud one before Sorting ended.
This doesn't read well too me, so for consideration, this is my suggestion to resolve #176.
@norwd norwd marked this pull request as draft December 28, 2024 06:48
@norwd norwd marked this pull request as ready for review December 28, 2024 06:52
@rrthomas
Copy link
Owner

Thanks for this suggestion. I don't think it helps: cheering is more enthusiastic than applauding, so Harry noticing that he's only applauded one person is uncharacteristically legalistic of him. It leaves the text in pretty much the same confusing place as before.

Also, I think your guess that the important thing is that Harry was trying to applaud Slytherins is spot on. Trying to form genuine relationships with Draco and others is a big theme of the book, and this is one of the earliest indicators of it.

@rrthomas rrthomas closed this Dec 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

minor problems
2 participants