[openbeos] Re: [Open-beos-cvs]CVS: current/src/add-ons/print
- From: Philippe Houdoin <philippe.houdoin@xxxxxxx>
- To: openbeos@xxxxxxxxxxxxx
- Date: Thu, 5 Aug 2004 17:19:21 +0200
Michael said:
> > I saw this checkin, and am concenred about the symantics of InitCheck()
> > here.
> >
> > I believe that InitCheck() everywhere else should be returning a
> > status_t.
>
> InitCheck is used in this source file only and it is not intended to
> be used anywhere else, so why should we worry about it?
Code consistency/cleaness?
In fact, all others print transport add-ons were already using InitCheck()
semantics, except for "USB Port" and "HP JetDirect".
Sadly, it take me two steps to fix it, and an Alan ;-).
- Philippe
Other related posts:
- » [openbeos] Re: [Open-beos-cvs]CVS: current/src/add-ons/print