Standardise EXPOSE and INTERNAL macros to PRIVATE
We should rename all our EXPOSE and INTERNAL macros to PRIVATE.
Then we can simplify the PRIVATE patterns in legacy/trac#32798 (moved) and legacy/trac#32522 (moved).
- Show closed items
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- teor changed milestone to %Tor: 0.4.3.x-final in legacy/trac
changed milestone to %Tor: 0.4.3.x-final in legacy/trac
Trac:
Parent Ticket: legacy/trac#32798 (moved)- teor added actualpoints::0.2 in Legacy / Trac component::core tor/tor in Legacy / Trac milestone::Tor: 0.4.3.x-final in Legacy / Trac owner::teor in Legacy / Trac parent::32798 in Legacy / Trac points::0.5 in Legacy / Trac priority::medium in Legacy / Trac resolution::fixed in Legacy / Trac reviewer::nickm in Legacy / Trac severity::normal in Legacy / Trac status::closed in Legacy / Trac type::task in Legacy / Trac labels
added actualpoints::0.2 in Legacy / Trac component::core tor/tor in Legacy / Trac milestone::Tor: 0.4.3.x-final in Legacy / Trac owner::teor in Legacy / Trac parent::32798 in Legacy / Trac points::0.5 in Legacy / Trac priority::medium in Legacy / Trac resolution::fixed in Legacy / Trac reviewer::nickm in Legacy / Trac severity::normal in Legacy / Trac status::closed in Legacy / Trac type::task in Legacy / Trac labels
I also standardised PRIVATE_ to PRIVATE.
The only unusual change is renaming CHANNEL_PRIVATE_ and TOR_CHANNEL_INTERNAL_ to CHANNEL_FILE_PRIVATE and CHANNEL_OBJECT_PRIVATE. Otherwise, they would have been ambiguous.
See my PR:
Trac:
Status: assigned to needs_review
Actualpoints: N/A to 0.2
Reviewer: N/A to nickmHere's how I searched for replacements:
$ grep -r '#.*if.*def.*[A-Z_]*\(INTERNAL\|EXPOSE\)[A-Z_]*' src | grep -v -e 'src/ext' -e 'include.am' src/lib/err/backtrace.h:#ifdef EXPOSE_CLEAN_BACKTRACE src/lib/err/backtrace.h:#endif /* defined(EXPOSE_CLEAN_BACKTRACE) */ ... src/core/or/channeltls.h:#ifdef TOR_CHANNEL_INTERNAL_ src/core/or/channeltls.h:#endif /* defined(TOR_CHANNEL_INTERNAL_) */ src/core/or/channel.h:#ifdef TOR_CHANNEL_INTERNAL_ src/core/or/channel.h:#endif /* defined(TOR_CHANNEL_INTERNAL_) */ ... src/feature/dirparse/routerparse.h:#ifdef EXPOSE_ROUTERDESC_TOKEN_TABLE $ grep -r '#.*if.*def.*PRIVATE_' src | grep -v -e 'src/ext' -e 'include.am' src/core/or/scheduler.h:#ifdef SCHEDULER_PRIVATE_ src/core/or/scheduler.h:#endif /* defined(SCHEDULER_PRIVATE_) */ src/core/or/channel.h:#ifdef CHANNEL_PRIVATE_ src/core/or/channel.h:#endif /* defined(CHANNEL_PRIVATE_) */
- Owner
lgtm; merged!
Trac:
Status: needs_review to closed
Resolution: N/A to fixed - Trac closed
closed
- Trac changed time estimate to 4h
changed time estimate to 4h
- Trac added 1h 36m of time spent
added 1h 36m of time spent
- Trac mentioned in issue legacy/trac#32798 (moved)
mentioned in issue legacy/trac#32798 (moved)
- Trac moved from legacy/trac#32818 (moved)
moved from legacy/trac#32818 (moved)
- Trac removed 1 deleted label
removed 1 deleted label