tech-userlevel archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: libcodecs(3), take 4



On Sat, Oct 02, 2010 at 02:22:22PM -0400, Thor Lancelot Simon wrote:
> On Sat, Oct 02, 2010 at 10:38:15AM +0100, David Laight wrote:
> > On Sat, Oct 02, 2010 at 08:13:25AM +0200, Alistair Crooks wrote:
> > > Hi Uebayashi-san,
> > > 
> > > Thanks for your feedback.
> > > 
> > > On Sat, Oct 02, 2010 at 02:42:02PM +0900, Masao Uebayashi wrote:
> > > > Usually libraries don't print error messages to stderr, nor exit.
> > > 
> > > That's true for some libraries, but not for all of them.  At the same
> > > time, printing error messages is far superior to the practise of
> > > returning enumerated error values (like regexp libraries), or stacking
> > > them up and vomiting them up at a later stage.
> 
> I don't think libraries should print to stderr, but I feel much, much
> more strongly that they must not exit the calling program.

Heh, don't believe everything you read on the internet

I'm not trying to say that people are trying to influence you unduly, but,
come on, please...

Regards,
Alistair
--- Begin Message ---
One thing I forgot...

On Sat, Oct 02, 2010 at 08:13:25AM +0200, Alistair Crooks wrote:
> On Sat, Oct 02, 2010 at 02:42:02PM +0900, Masao Uebayashi wrote:
> > Usually libraries don't print error messages to stderr, nor exit.

[23:18:10] agc@osx-vm1 ...codecs/dist/lib [5746] > grep exit *
Makefile:               && exit 0; \
Makefile:             exit 1;; \
Makefile:             cp -pR $(srcdir)/$$file $(distdir)$$dir || exit 1; \
Makefile:           cp -pR $$d/$$file $(distdir)$$dir || exit 1; \
Makefile:           || exit 1; \
Makefile.in:            && exit 0; \
Makefile.in:          exit 1;; \
Makefile.in:          cp -pR $(srcdir)/$$file $(distdir)$$dir || exit 1; \
Makefile.in:        cp -pR $$d/$$file $(distdir)$$dir || exit 1; \
Makefile.in:        || exit 1; \
[23:18:10] agc@osx-vm1 ...codecs/dist/lib [5747] >

Regards,
Alistair

--- End Message ---


Home | Main Index | Thread Index | Old Index