lazy_initialize_conv: Remove redundant call to init_valid_table,

because init_iconv calls it for us.

init_iconv: Add documentation about how this is reintialized when
configuration is reloaded.
(This used to be commit 0ee80a9018601fbba1c9558545de61cb16e94fac)

1 files changed, 6 insertions, 4 deletions

diff --git a/source3/lib/charcnv.c b/source3/lib/charcnv.c
index 42cba33483..dc270b94b7 100644
--- a/source3/lib/charcnv.c
+++ b/source3/lib/charcnv.c
@@ -67,14 +67,16 @@ static void lazy_initialize_conv(void)
 		initialized = True;
 		load_case_tables();
 		init_iconv();
-		init_valid_table();
 	}
 }
 
 /**
- Initialize iconv conversion descriptors.
-**/
-
+ * Initialize iconv conversion descriptors.
+ *
+ * This is called the first time it is needed, and also called again
+ * every time the configuration is reloaded, because the charset or
+ * codepage might have changed.
+ **/
 void init_iconv(void)
 {
 	int c1, c2;