Opened 13 years ago

Last modified 7 years ago

#2926 enhancement new

Various names in twisted.protocols.amp do not conform to coding standard recommendation for acronyms

Reported by: Glyph Owned by:
Priority: low Milestone:
Component: core Keywords:
Cc: itamarst, Thijs Triemstra Branch:


Quoth the coding standard:

"Acronyms should be capitalized in their entirety."

but AMP contains a few names which do not do this:

  • AmpBox
  • AmpError
    • RemoteAmpError
  • AmpList

These should all be very gradually deprecated and replaced with names such as AMPBox, AMPError, RemoteAMPError, AMPList.

Change History (4)

comment:1 Changed 13 years ago by itamarst

Cc: itamarst added

This seems silly; "amp" is fairly non-acronymy (has a vowel, is a real word.) The fact it also matches an acronym is nice, but I think in this case changing all uses to caps seems kinda pedantic. Compare to HTTP, that is unpronouncable, not a real word, and much more obviously an acronym.

comment:2 Changed 13 years ago by Glyph

It is a little silly, which is why I put it at "low" priority. However, the AMP class itself follows the coding standard, and consistency dictates that these should too.

comment:3 Changed 9 years ago by <automation>

Owner: Glyph deleted

comment:4 Changed 7 years ago by Thijs Triemstra

Cc: Thijs Triemstra added
Note: See TracTickets for help on using tickets.