You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@commons.apache.org by mt...@apache.org on 2009/08/22 16:27:38 UTC

svn commit: r806859 - /commons/sandbox/runtime/trunk/src/main/native/shared/ini.c

Author: mturk
Date: Sat Aug 22 14:27:38 2009
New Revision: 806859

URL: http://svn.apache.org/viewvc?rev=806859&view=rev
Log:
Add few comments

Modified:
    commons/sandbox/runtime/trunk/src/main/native/shared/ini.c

Modified: commons/sandbox/runtime/trunk/src/main/native/shared/ini.c
URL: http://svn.apache.org/viewvc/commons/sandbox/runtime/trunk/src/main/native/shared/ini.c?rev=806859&r1=806858&r2=806859&view=diff
==============================================================================
--- commons/sandbox/runtime/trunk/src/main/native/shared/ini.c (original)
+++ commons/sandbox/runtime/trunk/src/main/native/shared/ini.c Sat Aug 22 14:27:38 2009
@@ -153,7 +153,8 @@
 
 static char *expand_envars(char *s)
 {
-    /* TODO: Implement
+    /* TODO: Implement ${ENV} expansion.
+     * This should go in env.c
      */
     return s;
 }
@@ -163,6 +164,12 @@
     ini_section_free(root);
 }
 
+/*
+ * TODO:
+ * Use replacement file stdio api once when finished
+ * so we can use ucs2 file names and portable fgets.
+ * fopen, fgets and fclose need ACR replacement functions.
+ */
 ACR_DECLARE(ini_section_t *) ACR_IniLoadIni(JNIEnv *_E, const char *fname)
 {
     FILE *fp;