Alessandro Zummo writes:

> On Wed, 13 May 2015 08:28:24 +0900
> Olaf Meeuwissen <olaf.meeuwis...@avasys.jp> wrote:
>
>> Wouldn't the Right Thing to do be to change the backend to read its
>> configuration in sane_init()?  The SANE spec is pretty clear about the
>> fact that frontends do not have to call sane_get_devices() before they
>> call sane_open().
>
>  due to the way sanei_config_attach works, reading the config
>  file will trigger a scan of all the devices, which I would rather
>  avoid when the user specifies a device in sane_open().
>
>  the whole config system probably needs an overhaul, as
>  there's much duplicate code in a lot of drivers.

ACK.

>  we need something to parse the configuration on startup
>  and act on it at a later stage, but I don't have the time do
>  write it.
>
>  so if this patch works, it's the best I can do right now ;) 

I thought so much but it is better to be explicit about that, either in
the mail or better yet in the patch itself in the form of code comments
for example.

Thanks for the patch,
-- 
Olaf Meeuwissen, LPIC-2           FLOSS Engineer -- AVASYS CORPORATION
FSF Associate Member #1962               Help support software freedom
                 http://www.fsf.org/jf?referrer=1962

-- 
sane-devel mailing list: sane-devel@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/sane-devel
Unsubscribe: Send mail with subject "unsubscribe your_password"
             to sane-devel-requ...@lists.alioth.debian.org

Reply via email to