Josh Steadmon <steadmon@xxxxxxxxxx> writes: > From: Glen Choo <chooglen@xxxxxxxxxx> > > "struct config_options" is a disjoint set of options used by the config > parser (e.g. event listeners) and options used by config_with_options() > (e.g. to handle includes, choose which config files to parse). Can this sentence be reworded? In particular, "disjoint" is a word normally applied to two or more sets (meaning that they have no elements in common), but here it is used for only one. Everything else looks good, and the reasoning (some functions only use a subset of the fields, and this subset is easily explained conceptually as those related to parsing) makes sense.