[haiku-commits] Re: r34085 - haiku/trunk/src/system/kernel/fs

  • From: "Axel Dörfler" <axeld@xxxxxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Wed, 18 Nov 2009 13:14:26 +0100

"Axel Dörfler" <axeld@xxxxxxxxxxxxxxxx> wrote:
> "Ingo Weinhold" <ingo_weinhold@xxxxxx> wrote:
> > > > Log:
> > > > Added fs_read_attr().
> > > Why not just add fs_attr.cpp to kernel/lib/Jamfile as usual?
> > I just followed your shining example with the fs_*_info()
> > functions.
> > ;-)
> >
> > But seriously, I originally tried adding fs_attr.cpp , but instead
> > of
> > one undefined
> > fs_read_attr() this resulted in several undefined _kern_*()
> > symbols.
> > And then
> > my laziness won. :-)
> I guess the fs_{read|write}_info() function existence could have a
> similar reason :-)

Just had another look: the fs_{read|write}_info() functions are only
private. So AFAICT there was no shining example :-)
In the long run, it would be desirable to export all of fs_attr.h from
the kernel as well IMO.

Bye,
   Axel.


Other related posts: