Hi,
for me the better solution is to find the bug and eliminate it....
Phil
P.S. I don't have marks anymore only curses, so I consider I am almost in 
holidays !

Le Monday 12 June 2006 18:17, Harry Vennik a écrit :
> But that's not much of an argument here, because any solution will require
> patching libng or even its v4l2 driver.
>
> Anyway, by doing it Youness' way we will avoid an ugly thing like if(driver
> == pwc) { ..... } by adding a real feature. The only disadvantage being all
> reports about 'webcam not released when using v4l2' that we certainly will
> get.
>
> Harry
>
> Op zondag 11 juni 2006 23:44, schreef Philippe Valembois - Phil:
> > No without patching libng...
> > Phil
> >
> > Le Sunday 11 June 2006 22:45, Youness Alaoui a écrit :
> > > can't we make it use the driver you choose ? for example, the user can
> > > choose v4l or v4l2 from the list of devices, so if he chooses v4l, the
> > > v4l driver will be used, if he chooses v4l2, then the v4l2 driver will
> > > be used... and have a label (we already have one, right ?) for pwc
> > > users that would say "use the v4l device, not the v4l2 one"...
> > > is it possible ?
> > >
> > > KKRT
> > >
> > > On Sun, 11 Jun 2006 16:37:37 -0400, Harry Vennik <[EMAIL PROTECTED]>
> > >
> > > wrote:
> > > > Problem found.
> > > >
> > > > There are 2 work-arounds:
> > > > - If we somehow unset the V4L2_CAP_STREAMING flag for pwc, it will
> > > > work. - If we force pwc to v4l instead of v4l2 it will also work.
> > > >
> > > > The first one seems quite ugly to me, so IMO forcing to v4l is the
> > > > way to go.
> > > >
> > > > Op zondag 11 juni 2006 17:26, schreef Vivia Nikolaidou:
> > > >> On Sun, 11 Jun 2006, Max wrote:
> > > >> > I see at least one thing missing (imho) in the TODO-list and it is
> > > >> > a
> > > >>
> > > >> bug
> > > >>
> > > >> > fix for the pwc webcam's drivers. It was working perfectly before
> > > >> > but
> > > >>
> > > >> it
> > > >>
> > > >> > stopped working after 6511 (i checked). (i'am talking about the
> > > >> > "not released webcam").
> > > >>
> > > >> Thanx!! :)
> > > >>
> > > >> (BTW, bugfixes can be made between 2 RC releases)
> > > >>
> > > >> Here's the diff - hope you (or someone else) can find a fix for
> > > >> that:
> > > >>
> > > >> --- utils/linux/capture/capture.c       (revision 6510)
> > > >> +++ utils/linux/capture/capture.c       (revision 6511)
> > > >> @@ -480,6 +480,8 @@
> > > >>      captureItem->rgb_buffer =
> > > >> ng_malloc_video_buf(&captureItem->dev, &captureItem->fmt);
> > > >>    }
> > > >>
> > > >> +  captureItem->dev.v->startvideo(captureItem->dev.handle, 25, 1);
> > > >> +
> > > >>    Tcl_SetObjResult(interp,
> > > >> Tcl_NewStringObj(captureItem->captureName,-1));
> > > >>
> > > >>    return TCL_OK;
> > > >> @@ -506,6 +508,8 @@
> > > >>      return TCL_ERROR;
> > > >>    }
> > > >>
> > > >> +  capItem->dev.v->stopvideo(capItem->handle);
> > > >> +
> > > >>    // If a converter was used, close it and release the rgb_buffer
> > > >>    if (capItem->handle) {
> > > >>      ng_process_fini(capItem->handle);
> > > >> @@ -607,7 +611,7 @@
> > > >>    // - High resolution
> > > >>    // - Low resolution
> > > >>    for (dim_idx = resolution;;) {
> > > >> -    if ((capItem->image_data =
> > > >> capItem->dev.v->getimage(capItem->dev.handle)) == NULL) {
> > > >> +    if ((capItem->image_data =
> > > >> capItem->dev.v->nextframe(capItem->dev.handle)) == NULL) {
> > > >>  #   ifdef DEBUG
> > > >>        fprintf(stderr,"Capturing image failed at %d, %d\n",
> > > >> fmt.width, fmt.height);
> > > >>  #   endif
> > > >>
> > > >>
> > > >> _______________________________________________
> > > >> Amsn-devel mailing list
> > > >> Amsn-devel@lists.sourceforge.net
> > > >> https://lists.sourceforge.net/lists/listinfo/amsn-devel
> > > >
> > > > _______________________________________________
> > > > Amsn-devel mailing list
> > > > Amsn-devel@lists.sourceforge.net
> > > > https://lists.sourceforge.net/lists/listinfo/amsn-devel
> >
> > _______________________________________________
> > Amsn-devel mailing list
> > Amsn-devel@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/amsn-devel
>
> _______________________________________________
> Amsn-devel mailing list
> Amsn-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/amsn-devel


_______________________________________________
Amsn-devel mailing list
Amsn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/amsn-devel

Reply via email to