X-RT-Original-Encoding: utf-8
Content-Disposition: inline
X-Mailer: MIME-tools 5.508 (Entity 5.508)
Message-ID: <rt-4.4.1-1457-1506423510-432.46012-0-0@isc.org>
In-Reply-To: <rt-4.4.1-20250-1505706991-565.46012-0-0@isc.org>
X-RT-Interface: Web
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
References: <59bbf92280b75_2176717334785c@ss1435.mail>
 <RT-Ticket-46012@isc.org>
 <rt-4.4.1-20250-1505706991-565.46012-0-0@isc.org>
Content-Transfer-Encoding: binary
RT-Send-CC:
Content-Length: 443

I prepared an alternative approach in rt46012_michal.  It slightly
refactors the code in configure_rpz() so that it reads a bit more nicely
by avoiding a double #ifdef and a triple dnsrps_enabled check while
retaining identical code flow.  As a bonus, it surrounds conf_dnsrps*
with an #ifdef to prevent a compiler warning when building without
DNSRPS support.  If you like it, feel free to use it.  Otherwise, your
fix is good to go as well.