git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] doc: replace jargon word "impact" with "effect"/"affect"

From
Felipe Contreras <felipe.contreras@gmail.com>
Date
Jun 13, 2021, 14:06 UTC
Message-ID
<60c610f04b288_41f2b208ce@natae.notmuch>
In-Reply-To
<20210613114007.GF8544@kitsune.suse.cz>
Michal Suchánek wrote:
> On Sat, Jun 12, 2021 at 06:13:02PM -0500, Varun Varada wrote:
> > On Thu, 27 May 2021 at 09:35, Michal Suchánek <msuchanek@suse.de> wrote:
Show 5 quoted lines
> > > That's only opinion, not evidence.
> > 
> > Hence, the dictionary definitions, which are evidence. If people are
> 
> The dictionaries provide definition, not usage guidelines.
What do you think definitions are?
A definition is there because many people use the word that way.
Show 9 quoted lines
> > Yes, there essentially is. It's called a dictionary. If you don't
> > respect the value of dictionaries, you're tacitly claiming that anyone
> 
> I don't consider opinions tangentially related to dictionary content a
> proof of anything.
> 
> Also it has been pointed out that dictionaries don't agree on the
> precise definition - hence no final authority. The laguage use varies,
> and dictionaries also reflect that. Even the use of word 'impact'.

You need to understand what dictionaries do: they describe current usage.

If some dictionaries say "impact" and "affect" are not 100% synonyms, that means some people don't consider "impact" and "affect" to be synonymns.

Period.
Show 11 quoted lines
> > > This will bring in reviews that focus on hairsplitting when the
> > > formulation with 'impact' reads better than 'effect' and where the
> > > change does not make it read any better so it should not be changed.
> > >
> > > It also brings in reviews of the sort that simply say that use of
> > > 'impact' is OK, and there is no need to change.
> > 
> > That's an "if". This, however, is a situation where multiple people
> 
> We already received such reviews as response to your patch. It's not
> what-if.

In case you haven't been following this thread closely, you are the only person that says the use of "impact" is OK. One person said "impact" was OK for him, but he didn't say anything of anybody else. Another person asked if they were synonyms. That's it.

I say Varun should resend the patch separate from all other patches, explain why they aren't synonyms and mention for the record that one person objects to the change.

It's OK to merge patches where one person objects.
-- 
Felipe Contreras
Previous: Michal SuchánekNext: Michal Suchánek
Message 54 of 68 in “doc: replace jargon word "impact" with "effect"/"affect"”
  1. doc: replace jargon word "impact" with "effect"/"affect"Varun Varada, Apr 5, 2021
  2. Michal SuchánekApr 6, 2021
  3. Varun VaradaApr 6, 2021
  4. Jeff KingApr 6, 2021
  5. Varun VaradaApr 7, 2021
  6. Varun VaradaApr 28, 2021
  7. Michal SuchánekApr 28, 2021
  8. Varun VaradaApr 28, 2021
  9. Michal SuchánekApr 28, 2021
  10. Varun VaradaApr 30, 2021
  11. Michal SuchánekApr 30, 2021
  12. Varun VaradaMay 10, 2021
  13. Michal SuchánekMay 10, 2021
  14. Varun VaradaMay 10, 2021
  15. Michal SuchánekMay 11, 2021
  16. Junio C HamanoMay 11, 2021
  17. Felipe ContrerasMay 12, 2021
  18. Felipe ContrerasMay 12, 2021
  19. Felipe ContrerasMay 12, 2021
  20. Felipe ContrerasMay 12, 2021
  21. Felipe ContrerasMay 12, 2021
  22. Felipe ContrerasMay 12, 2021
  23. Felipe ContrerasMay 11, 2021
  24. Michal SuchánekMay 11, 2021
  25. Varun VaradaMay 11, 2021
  26. Felipe ContrerasMay 12, 2021
  27. Michal SuchánekMay 12, 2021
  28. Felipe ContrerasMay 12, 2021
  29. Michal SuchánekMay 12, 2021
  30. Felipe ContrerasMay 12, 2021
  31. Michal SuchánekMay 12, 2021
  32. Michal SuchánekMay 12, 2021
  33. Felipe ContrerasMay 12, 2021
  34. Michal SuchánekMay 12, 2021
  35. Robert P. J. DayMay 12, 2021
  36. Kerry, RichardMay 12, 2021
  37. Varun VaradaMay 12, 2021
  38. Michal SuchánekMay 12, 2021
  39. Felipe ContrerasMay 12, 2021
  40. Michal SuchánekMay 12, 2021
  41. Felipe ContrerasMay 12, 2021
  42. Michal SuchánekMay 13, 2021
  43. Felipe ContrerasMay 13, 2021
  44. Robert CoupMay 13, 2021
  45. Michal SuchánekMay 13, 2021
  46. Felipe ContrerasMay 13, 2021
  47. Varun VaradaMay 26, 2021
  48. Michal SuchánekMay 27, 2021
  49. Felipe ContrerasMay 27, 2021
  50. Michal SuchánekMay 27, 2021
  51. Felipe ContrerasMay 27, 2021
  52. Varun VaradaJun 12, 2021
  53. Michal SuchánekJun 13, 2021
  54. Felipe ContrerasJun 13, 2021
  55. Michal SuchánekJun 13, 2021
  56. Felipe ContrerasJun 13, 2021
  57. Varun VaradaMay 12, 2021
  58. Felipe ContrerasMay 13, 2021
  59. Felipe ContrerasMay 12, 2021
  60. Felipe ContrerasMay 11, 2021
  61. Michal SuchánekMay 11, 2021
  62. Felipe ContrerasMay 12, 2021
  63. Michal SuchánekMay 12, 2021
  64. Felipe ContrerasMay 12, 2021
  65. Varun VaradaMay 12, 2021
  66. Philip OakleyMay 13, 2021
  67. Varun VaradaMay 26, 2021
  68. Philip OakleyMay 27, 2021

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.