Hi Paul,

> here's the place where the new "containsnul" is used, for CT_TEXT and
> CT_APPLICATION.
...
>         case CT_TEXT:
>             if (contains8bit && !containsnul && !linelen && !linespace && 
> !checksw)
>                 ct->c_encoding = CE_8BIT;
>             else if (contains8bit || containsnul || linelen || linespace || 
> checksw)
>                 ct->c_encoding = CE_QUOTED;
>             else
>                 ct->c_encoding = CE_7BIT;
>             break;

The duplication of contains8bit seems overly complex.

    case CT_TEXT:
        if (contains8bit)
            ct->c_encoding = (containsnul || linelen || linespace || checksw) ?
                CE_QUOTED : CE_8BIT;
        else
            ct->c_encoding = CE_7BIT;

Cheers, Ralph.

_______________________________________________
Nmh-workers mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/nmh-workers

Reply via email to