David Muir Sharnoff wrote:
+    if ($status) {
+    # this section needs to be kept in sync with the cleanup_stat_map
+    # array found in Postfix source file src/global/cleanup_strerror.c
+    # which in turn uses constants defined in src/global/cleanup_user.h

It's extremely icky to have Perl code that relies on some other project's .c and .h files like that. Do you know how frequent changes are made to that array? Is there any way that you can provide code to regenerate this block directly from the Postfix source (and perhaps store the result in a DATA block at the end of the plugin, rather than inline)?

If the Postfix array is infrequently updated, then I can see applying your patch, but I'd rather do something that will be easier to maintain...

John

Reply via email to