[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kopete-devel
Subject:    Re: [kopete-devel] AVDevices which respond,
From:       Matt Rogers <mattr () kde ! org>
Date:       2008-12-29 4:24:08
Message-ID: 200812282224.09061.mattr () kde ! org
[Download RAW message or body]

On Sunday 28 December 2008 08:30:25 pm Alan Jones wrote:
> Hi All,
>
> I guess I should do a quick intro, this being my first email to the list.
> I'm Alan, have been coding for a while and wanted to help out with KDE
> somehow. I was told kopete is in need so I've been starting on some bugs.
>
> I've put a fix in place for 161536, but I think it could do with some
> discussion on the list to prevent it even going down the path it does to
> get
> there. Some (all?)
> webcams still respond to some of the queries kopete asks while another
> application is using them. This was causing kopete to try and read the
> buffer while it is empty and crash.
>
> It would be better if kopete realized the device was unavailable, so I
> didn't know
> whether modifying VideoDevice::isOpen() and VideoDevice::initDevice() to
> perform this check as well would be a good solution. My concerns were where
> this is best addressed, as the fix I've put in doesn't feel ideal. Removing
> the device didn't
> make sense either as even though it's not available at that time they
> may want it
> again later during their current kopete session and it may be really
> available at that
> time. I didn't know whether they've be forced to go back into
> configuration to restore
> it which seemed less than ideal.
>
> Anyhow - just thought I'd raise it and see what people's thoughts were.
>
> Cheers,
>
> Alan.

I agree with all of this. Please post a patch when you can. 

Thanks for working on our video device support!
-- 
Matt
_______________________________________________
kopete-devel mailing list
kopete-devel@kde.org
https://mail.kde.org/mailman/listinfo/kopete-devel
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic