Hi Grant,

On 5/20/19 11:44 PM, Grant Taylor via bind-users wrote:
On 5/20/19 4:34 AM, Matthijs Mekking wrote:
* It will make the code much easier to maintain, which is beneficial for users too since that will mean in general less bugs, easier to find bugs, and easier to extend it with new features.

Drive by 2¢ comment:

Is the existing DLV code causing a problem or otherwise breaking something?

Not actively, but in general it adds more corner cases, which make it harder to investigate potential bugs in validation behavior.

How much easier will removing the DLV code make maintaining the rest of the code base?

Hard to say, but quoting my colleague "about 50%".


Is the existing DLV code preventing doing something else that is desired?

Not preventing, but it is something that we need to take into account every time we touch the validation code, and so there is a maintenance cost.


IMHO if the code is sitting there and not actively causing problems, despite being unsightly, then I'd be inclined to leave it.  If it's anything more than unsightly, I'd pontificate removing it.

Thanks for your input.

Best regards,

Matthijs






_______________________________________________
Please visit https://lists.isc.org/mailman/listinfo/bind-users to unsubscribe 
from this list

bind-users mailing list
bind-users@lists.isc.org
https://lists.isc.org/mailman/listinfo/bind-users

_______________________________________________
Please visit https://lists.isc.org/mailman/listinfo/bind-users to unsubscribe 
from this list

bind-users mailing list
bind-users@lists.isc.org
https://lists.isc.org/mailman/listinfo/bind-users

Reply via email to