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.