On 12/29/14 at 02:07pm, Allan McRae wrote:
> On 27/04/14 09:57, Andrew Gregory wrote:
> > Signed-off-by: Andrew Gregory <[email protected]>
> > ---
>
> Why? This seems a reasonable restriction.
To move the error handling out of the ini parser. alpm will already
reject an empty db name. I'll add a commit message to that effect.
> > src/pacman/ini.c | 7 -------
> > 1 file changed, 7 deletions(-)
> >
> > diff --git a/src/pacman/ini.c b/src/pacman/ini.c
> > index 5c5232d..a8c4c04 100644
> > --- a/src/pacman/ini.c
> > +++ b/src/pacman/ini.c
> > @@ -81,13 +81,6 @@ int parse_ini(const char *file, ini_parser_fn cb, void
> > *data)
> >
> > if(line[0] == '[' && line[line_len - 1] == ']') {
> > char *name;
> > - /* only possibility here is a line == '[]' */
> > - if(line_len <= 2) {
> > - pm_printf(ALPM_LOG_ERROR, _("config file %s,
> > line %d: bad section name.\n"),
> > - file, linenum);
> > - ret = 1;
> > - goto cleanup;
> > - }
> > /* new config section, skip the '[' */
> > name = strdup(line + 1);
> > name[line_len - 2] = '\0';
> >
