Nick Mathewson authored 5 years ago and David Goulet committed 5 years ago

Here we make it clear we're only looking at listable variable names,
not at whether the variables themselves are gettable.

Also, remove an extraneous h.

(This commit is not a fixup, because of rebase conflicts.)

Nick Mathewson authored 5 years ago and David Goulet committed 5 years ago

(This option tests our existing behavior, not necessarily the most
sensible behavior.)

Nick Mathewson authored 5 years ago and David Goulet committed 5 years ago

The old names remain as #defines that cause variables to get one or
more flags.

Now every flag-testing function in confparse.c tests exactly one flag.

Nick Mathewson authored 5 years ago and David Goulet committed 5 years ago

Since the flags are now stored with compatible numbering, we can
just OR them together and see whether the flag we want is in the
result.

(Net code removal!)

Nick Mathewson authored 5 years ago and David Goulet committed 5 years ago

Using a bitfield here will enable us to unify the var_type_def_t flags
with the config_var_t flags.

(This commit does not yet do that unification, and does not yet
rename or refactor any flags.  It only changes booleans into bits.)

Nick Mathewson authored 5 years ago and David Goulet committed 5 years ago

Previously they checked the individual flags inside var_type_def_t;
now they call the appropriate var_type_is_*() functions.

(These functions will be removed entirely by the end of this branch.)

Nick Mathewson authored 5 years ago and David Goulet committed 5 years ago

Don't use "derived" directly, but check its implications for listing
and copying.

Nick Mathewson authored 5 years ago and David Goulet committed 5 years ago

We had though to make all obsolete and invisible variables
ungettable, so that GETCONF would reject them.  But it turns out
that this isn't the current behavior of GETCONF with those
variables.  So for now, I'm leaving the current behavior unchanged.
(See ticket 31647 for a proposal to change the behavior.)

This change replaces the higher-level property check to match our
intended flag factoring.

Resolved modified/deleted conflict on changes/bug30649 by deleting
the file.

tor-github/pr/1282 modifies ChangeLog and ReleaseNotes instead of
changes/bug30649. We cherry-picked it to master, as well as merging
to release-0.4.1.

tor-github/pr/1283, with the following changes:
* cherry-pick the merge commit in 1283 on top of...
* tor-github/pr/1174, but with the last commit re-worded to remove
    the fixup, because fixups break our push rules.

This is an "ours" merge, except for the bugfix version change in
changes/bug30649.