get rid of redundant should_record_bridge_info() call in options_act()
In options_act() we have
if ((!old_options || !old_options->EntryStatistics) &&
options->EntryStatistics && !should_record_bridge_info(options)) {
But right above that we have
/* Only collect other relay-only statistics on relays. */
if (!public_server_mode(options)) {
options->CellStatistics = 0;
options->EntryStatistics = 0;
So the only way EntryStatistics could be non-zero at that lower point is if public_server_mode() is true.
So the check to !should_record_bridge_info() will always be true, and is redundant to check.