[tor-bugs] #29223 [Core Tor/Tor]: List canonical abbreviations to use in Tor functions and identifiers
Tor Bug Tracker & Wiki
blackhole at torproject.org
Wed Jan 30 16:12:08 UTC 2019
#29223: List canonical abbreviations to use in Tor functions and identifiers
------------------------------+--------------------
Reporter: nickm | Owner: (none)
Type: task | Status: new
Priority: Medium | Milestone:
Component: Core Tor/Tor | Version:
Severity: Normal | Keywords:
Actual Points: | Parent ID:
Points: 0.5 | Reviewer:
Sponsor: |
------------------------------+--------------------
We often don't abbreviate things in our function names and global
identifiers. This leads to beautiful things like
"router_get_mutable_consensus_status_by_descriptor_digest" and
"signed_descs_update_status_from_consensus_networkstatus" and
"networkstatus_consensus_can_use_multiple_directories".
What's more, when we do abbreviate, we often do so inconsistently: cfg
versus config, con vs conn, orconn vs or_conn, and so on.
We should specify which abbreviations we want to standardize, and converge
on those.
It is most important to do this for identifiers that are visible across
modules, followed by identifiers that are global within a module. I don't
think we need to do this for local variables.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/29223>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tor-bugs
mailing list