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.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8858#note_114592