[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