add a semantic patch to prevent (most) future uses of the
CHECK and RETERR patterns.
--- /dev/null
+@@
+expression E1;
+@@
+
+- result = E1;
+- if (result != ISC_R_SUCCESS) {
+- goto cleanup;
+- }
++ CHECK(E1);
+
+@@
+expression E1;
+isc_result_t result;
+@@
+
+- result = E1;
+- if (result != ISC_R_SUCCESS) {
+- return result;
+- }
++ RETERR(E1);
+
+@@
+expression E1;
+@@
+
+- result = E1;
+- if (result != ISC_R_SUCCESS) {
+- CHECK(result);
+- }
++ CHECK(E1);