Re: [QCAP] Final patch

2005-05-20 Thread Alexandre Julliard
Maarten Lankhorst <[EMAIL PROTECTED]> writes: > + > + if (!S_ISCHR (st.st_mode)) { > + ERR("%s: Not a device\n", device); > + CoTaskMemFree(capBox); > + return E_FAIL; > + } > > > > >Same here. > > > No, /dev/video* should never be an ordinary file,

Re: [QCAP] Final patch

2005-05-20 Thread Maarten Lankhorst
Dimi Paun wrote: On Fri, 2005-05-20 at 12:26 +0200, Maarten Lankhorst wrote: What do you suggest then ... fprintf(stderr? Absolutely not. :) But there are other options available: TRACE/WARN/FIXME. There's also MESSAGE, but it should be used *very* little. Also remember that most people wil

Re: [QCAP] Final patch

2005-05-20 Thread Dimi Paun
On Fri, 2005-05-20 at 12:26 +0200, Maarten Lankhorst wrote: > What do you suggest then ... fprintf(stderr? Absolutely not. :) But there are other options available: TRACE/WARN/FIXME. There's also MESSAGE, but it should be used *very* little. Also remember that most people will not see MESSAGE()s a

Re: [QCAP] Final patch

2005-05-20 Thread Dimi Paun
On Fri, 2005-05-20 at 10:37 +0200, Andreas Mohr wrote: > > > + ERR("Tinkerer detected? Thou shalt not define HAVE_V4L2 > \n"); > > > + CoTaskMemFree(capBox); > > > + close(capBox->fd); > > > + return E_FAIL; > > > > This is a FIXME. > It's not. His comments indicate that he has

Re: [QCAP] Final patch

2005-05-20 Thread Andreas Mohr
Hi, On Thu, May 19, 2005 at 09:20:25PM -0400, Dimi Paun wrote: > On Fri, 2005-05-20 at 00:42 +0200, Maarten Lankhorst wrote: > > + > > + capBox->fd = open(device, O_RDWR | O_NONBLOCK); > > + if (capBox->fd == -1) { > > + ERR("%s: Failed to open: %s\n", device, strerror(errno)); > > +

Re: [QCAP] Final patch

2005-05-20 Thread Maarten Lankhorst
Dimi Paun wrote: On Fri, 2005-05-20 at 00:42 +0200, Maarten Lankhorst wrote: m3h, v4l driver for vfwcapture.. please leave the ERR notice on initialisation intact, thank you :) I think you've overusing ERR(). ERR() should be called to signal an internal error such as inconsistent state, no

Re: [QCAP] Final patch

2005-05-19 Thread Dimi Paun
On Fri, 2005-05-20 at 00:42 +0200, Maarten Lankhorst wrote: > m3h, v4l driver for vfwcapture.. please leave the ERR notice on > initialisation intact, thank you :) I think you've overusing ERR(). ERR() should be called to signal an internal error such as inconsistent state, not something that the