diff options
| author | Nick Mathewson <nickm@torproject.org> | 2010-09-30 14:58:27 -0400 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2010-10-01 18:14:27 -0400 |
| commit | 45f1e4d5ee7b6e2308655628d18a67d330d9b624 (patch) | |
| tree | e78de04bf2888e7ca2abe5b80f91ae01949cd795 /src/or/or.h | |
| parent | 26e897420e07611e0b2c10b28202c388eea4bd6b (diff) | |
| download | tor-45f1e4d5ee7b6e2308655628d18a67d330d9b624.tar.gz tor-45f1e4d5ee7b6e2308655628d18a67d330d9b624.zip | |
Rename routerstatus_t.is_running to is_flagged_running
This was the only flag in routerstatus_t that we would previously
change in a routerstatus_t in a consensus. We no longer have reason
to do so -- and probably never did -- as you can now confirm more
easily than you could have done by grepping for is_running before
this patch.
The name change is to emphasize that the routerstatus_t is_running
flag is only there to tell you whether the consensus says it's
running, not whether it *you* think it's running.
Diffstat (limited to 'src/or/or.h')
| -rw-r--r-- | src/or/or.h | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/or/or.h b/src/or/or.h index d9be013332..670cf7dd40 100644 --- a/src/or/or.h +++ b/src/or/or.h @@ -1586,7 +1586,11 @@ typedef struct routerstatus_t { unsigned int is_exit:1; /**< True iff this router is a good exit. */ unsigned int is_stable:1; /**< True iff this router stays up a long time. */ unsigned int is_fast:1; /**< True iff this router has good bandwidth. */ - unsigned int is_running:1; /**< True iff this router is up. */ + /** True iff this router is called 'running' in the consensus. We give it + * this funny name so that we don't accidentally use this bit as a view of + * whether we think the router is *currently* running. If that's what you + * want to know, look at is_running in node_t. */ + unsigned int is_flagged_running:1; unsigned int is_named:1; /**< True iff "nickname" belongs to this router. */ unsigned int is_unnamed:1; /**< True iff "nickname" belongs to another * router. */ |