Content-Transfer-Encoding: binary
Message-ID: <rt-4.4.1-21730-1512042858-793.46718-0-0@isc.org>
X-RT-Original-Encoding: utf-8
X-RT-Interface: Web
X-Mailer: MIME-tools 5.508 (Entity 5.508)
References: <RT-Ticket-46718@isc.org>
 <rt-4.4.1-8816-1511877037-1497.46718-4-0@isc.org>
 <rt-4.4.1-8816-1511966481-1199.46718-4-0@isc.org>
 <rt-4.4.1-8816-1511968062-1074.46718-4-0@isc.org>
 <rt-4.4.1-15072-1511979113-1036.46718-4-0@isc.org>
 <7c847a1e-32fb-6b09-9886-1ea5ec7f7015@isc.org>
 <rt-4.4.1-20585-1511992308-530.46718-0-0@isc.org>
 <rt-4.4.1-15072-1512028872-133.46718-0-0@isc.org>
Content-Type: text/plain; charset="utf-8"
In-Reply-To: <rt-4.4.1-15072-1512028872-133.46718-0-0@isc.org>
MIME-Version: 1.0
Content-Disposition: inline
RT-Send-CC:
Content-Length: 843

On Thu Nov 30 08:01:12 2017, michal wrote:
> Pushed a commit which causes callback parameters to be passed explicitly
> instead of using a va_list.  As this is one of the options Ondřej
> suggested in his last message, I think rt46718 will be good to go once
> someone takes one final look at it.

A really minor nit, I would personally prefer to replace:

if (!commit || result != ISC_R_SUCCESS) {

with

if ((commit == NULL)[1] || (result != ISC_R_SUCCESS)[2]) {

with this explanation:

[1] to be consistent with rest of code where we use 'text != NULL' (as an example from this commit

[2] And the parentheses while not necessary just contributes to readability and prevents future mistakes.

~~~

But no strong feelings, just an observation.  It could be merged as is even without this change.  So, go ahead and merge at your discretion.