From 6334b0965307b89e37b21c59220c3ffd80a969c3 Mon Sep 17 00:00:00 2001
From: Tom Eastep <teastep@shorewall.net>
Date: Tue, 26 Mar 2013 14:30:42 -0700
Subject: [PATCH] Add a comment about why the 'id' member of builtin tables is
 initialized in process_providers()

Signed-off-by: Tom Eastep <teastep@shorewall.net>
---
 Shorewall/Perl/Shorewall/Providers.pm | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Shorewall/Perl/Shorewall/Providers.pm b/Shorewall/Perl/Shorewall/Providers.pm
index 36cb6d6b0..a8340757e 100644
--- a/Shorewall/Perl/Shorewall/Providers.pm
+++ b/Shorewall/Perl/Shorewall/Providers.pm
@@ -1338,7 +1338,9 @@ sub process_providers( $ ) {
 
     our $providers = 0;
     our $pseudoproviders = 0;
-
+    #
+    # We defer initialization of the 'id' member until now so that the setting of USE_RT_NAMES will have been established.
+    #
     unless ( $config{USE_RT_NAMES} ) {
 	for ( values %providers ) {
 	    $_->{id} = $_->{number};