Attachment 828357 Details for Bug 734434 – Try to merge with popt 1.16

[patch] Try to merge with popt 1.16

popt-1.16-help.patch (text/plain), 3.22 KB, created by Robert Scheck on 2013-11-24 16:11:55 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Robert Scheck

Created: 2013-11-24 16:11:55 UTC

Size: 3.22 KB

patch

obsolete

>--- popt-1.16/popthelp.c			2009-08-28 02:06:33.000000000 +0200
>+++ popt-1.16/popthelp.c.help			2013-11-24 16:38:44.000000000 +0100
>@@ -527,8 +527,11 @@
>     if (opt != NULL)
>     while (opt->longName || opt->shortName || opt->arg) {
> 	if (poptArgType(opt) == POPT_ARG_INCLUDE_TABLE) {
>-	    if (opt->arg)	/* XXX program error */
>-	        len = maxArgWidth(opt->arg, translation_domain);
>+	    void * arg = opt->arg;
>+	    /* XXX sick hack to preserve pretense of ABI. */
>+	    if (arg == poptHelpOptions) arg = poptHelpOptionsI18N;
>+	    if (arg)	/* XXX program error */
>+		len = maxArgWidth(arg, translation_domain);
> 	    if (len > max) max = len;
> 	} else if (!F_ISSET(opt, DOC_HIDDEN)) {
> 	    len = sizeof("  ")-1;
>@@ -619,9 +622,12 @@
> 
>     if (table != NULL)
>     for (opt = table; opt->longName || opt->shortName || opt->arg; opt++) {
>+	void * arg = opt->arg;
> 	if (poptArgType(opt) != POPT_ARG_INCLUDE_TABLE)
> 	    continue;
>-	sub_transdom = getTableTranslationDomain(opt->arg);
>+	/* XXX sick hack to preserve pretense of ABI. */
>+	if (arg == poptHelpOptions) arg = poptHelpOptionsI18N;
>+	    sub_transdom = getTableTranslationDomain(arg);
> 	if (sub_transdom == NULL)
> 	    sub_transdom = translation_domain;
> 	    
>@@ -631,7 +637,7 @@
> 	if (opt->descrip)
> 	    xx = POPT_fprintf(fp, "\n%s\n", D_(sub_transdom, opt->descrip));
> 
>-	singleTableHelp(con, fp, opt->arg, columns, sub_transdom);
>+	singleTableHelp(con, fp, arg, columns, sub_transdom);
>     }
> }
> 
>@@ -808,22 +814,25 @@
> 	    translation_domain = (const char *)opt->arg;
> 	} else
> 	if (poptArgType(opt) == POPT_ARG_INCLUDE_TABLE) {
>+	    void * arg = opt->arg;
>+	    /* XXX sick hack to preserve pretense of ABI. */
>+	    if (arg == poptHelpOptions) arg = poptHelpOptionsI18N;
> 	    if (done) {
> 		int i = 0;
> 		if (done->opts != NULL)
> 		for (i = 0; i < done->nopts; i++) {
> 		    const void * that = done->opts[i];
>-		    if (that == NULL || that != opt->arg)
>+		    if (that == NULL || that != arg)
> 			/*@innercontinue@*/ continue;
> 		    /*@innerbreak@*/ break;
> 		}
> 		/* Skip if this table has already been processed. */
>-		if (opt->arg == NULL || i < done->nopts)
>+		if (arg == NULL || i < done->nopts)
> 		    continue;
> 		if (done->opts != NULL && done->nopts < done->maxopts)
>-		    done->opts[done->nopts++] = (const void *) opt->arg;
>+		    done->opts[done->nopts++] = (const void *) arg;
> 	    }
>-	    columns->cur = singleTableUsage(con, fp, columns, opt->arg,
>+	    columns->cur = singleTableUsage(con, fp, columns, arg,
> 			translation_domain, done);
> 	} else
> 	if ((opt->longName || opt->shortName) && !F_ISSET(opt, DOC_HIDDEN)) {
>@@ -864,9 +873,13 @@
> 	    if (!strchr(s, opt->shortName) && isprint((int)opt->shortName)
> 	     && opt->shortName != ' ')
> 		s[strlen(s)] = opt->shortName;
>-	} else if (poptArgType(opt) == POPT_ARG_INCLUDE_TABLE)
>-	    if (opt->arg)	/* XXX program error */
>-		len = showShortOptions(opt->arg, fp, s);
>+	} else if (poptArgType(opt) == POPT_ARG_INCLUDE_TABLE) {
>+	    void * arg = opt->arg;
>+	    /* XXX sick hack to preserve pretense of ABI. */
>+	    if (arg == poptHelpOptions) arg = poptHelpOptionsI18N;
>+	    if (arg)	/* XXX program error */
>+		len = showShortOptions(arg, fp, s);
>+	}
>     } 
> 
>     /* On return to top level, print the short options, return print length. */

Actions: View | Diff

Attachments on bug 734434: 572052 | 828357 | 846913