diff options
| author | Dmitri Pal <dpal@redhat.com> | 2009-12-07 09:24:32 -0500 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2009-12-10 17:20:26 -0500 |
| commit | 150431bbb08bfac7ec0bd751a405b91f79646640 (patch) | |
| tree | 05a5b4d09bf9c99e0278ff90cd9a81998c021c7b /ini | |
| parent | f1e963ecdfde201e80251462e4e7737a0ef00eae (diff) | |
| download | ding-libs-150431bbb08bfac7ec0bd751a405b91f79646640.tar.gz ding-libs-150431bbb08bfac7ec0bd751a405b91f79646640.tar.xz ding-libs-150431bbb08bfac7ec0bd751a405b91f79646640.zip | |
INI: Cleaning FIXME comments.
Added configurable key length.
Changed comments for the functions that
are currently not used and reserved for
future functionality.
Diffstat (limited to 'ini')
| -rw-r--r-- | ini/configure.ac | 2 | ||||
| -rw-r--r-- | ini/ini_config.c | 16 | ||||
| -rw-r--r-- | ini/ini_config.h | 5 |
3 files changed, 16 insertions, 7 deletions
diff --git a/ini/configure.ac b/ini/configure.ac index c599124..ef8f0de 100644 --- a/ini/configure.ac +++ b/ini/configure.ac @@ -21,5 +21,7 @@ AC_ARG_ENABLE([trace], [trace_level="0"]) AS_IF([test ["$trace_level" -gt "0"] -a ["$trace_level" -lt "8"] ],[AC_SUBST([TRACE_VAR],["-DTRACE_LEVEL=$trace_level"])]) +AC_DEFINE([MAX_KEY], [1024], [Max length of the key in the INI file.]) + AC_CONFIG_FILES([Makefile ini_config.pc]) AC_OUTPUT diff --git a/ini/ini_config.c b/ini/ini_config.c index bd4fa79..6766dfa 100644 --- a/ini/ini_config.c +++ b/ini/ini_config.c @@ -110,11 +110,15 @@ const char *parsing_error_str(int parsing_error) return str_error[parsing_error-1]; } -/* Function to return grammar error */ +/* Function to return grammar error. + * This functions is currently not used. + * It is planned to be used by the INI + * file grammar parser. + */ const char *grammar_error_str(int grammar_error) { const char *placeholder= _("Unknown grammar error."); - /* FIXME - this is a temporary placeholder !!!! */ + /* THIS IS A TEMPORARY PLACEHOLDER !!!! */ const char *str_error[] = { _(""), _(""), _(""), @@ -131,11 +135,15 @@ const char *grammar_error_str(int grammar_error) return str_error[grammar_error-1]; } -/* Function to return validation error */ +/* Function to return validation error. + * This functions is currently not used. + * It is planned to be used by the INI + * file grammar validator. + */ const char *validation_error_str(int validation_error) { const char *placeholder= _("Unknown validation error."); - /* FIXME - this is a temporary placeholder !!!! */ + /* THIS IS A TEMPORARY PLACEHOLDER !!!! */ const char *str_error[] = { _(""), _(""), _(""), diff --git a/ini/ini_config.h b/ini/ini_config.h index 5ae4981..f7e0415 100644 --- a/ini/ini_config.h +++ b/ini/ini_config.h @@ -26,6 +26,7 @@ #include <limits.h> #include <stdio.h> #include "collection.h" +#include "config.h" /* Name of the default (missing section in the INI file */ #define INI_DEFAULT_SECTION "default" @@ -68,9 +69,7 @@ -/* Internal sizes */ -/* FIXME - make them configurable via config.h */ -#define MAX_KEY 1024 +/* Internal sizes. MAX_KEY is defined in config.h */ #define MAX_VALUE PATH_MAX #define BUFFER_SIZE MAX_KEY + MAX_VALUE + 3 |