Re: [PATCH 0/1] MR8858: sane.ds: Adding saving scanner settings to the registry

[Esme Povirk (＠madewokherd)]

Esme Povirk (@madewokherd) commented about dlls/sane.ds/ui.c:
> -
>        BOOL is_correct = FALSE;
> -
>        for (j=1; j<=count; j++)
>        {
> -
>            if (opt.constraint.word_list[j] == val)
> -
>            {
> -
>                is_correct = TRUE;
> -
>            }
>            sprintf(buffer, "%d", opt.constraint.word_list[j]);
>            SendMessageA(control, CB_ADDSTRING, 0, (LPARAM)buffer);
>        }
> -
>        if (is_exist && is_correct)
> -
>            sane_option_set_value( i, &val, NULL );
> -
> -
>        if (!is_correct)
> -
>            ERR("%s=%d is incorrect. The default value is set!\n", title, val);

I don't think we should print this if the setting doesn't exist.