[haiku-commits] Re: r40301 - in haiku/trunk: headers/os/package headers/private headers/private/package headers/private/package/hpkg src/add-ons/kernel/file_systems/packagefs ...

  • From: "Axel Dörfler" <axeld@xxxxxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Fri, 28 Jan 2011 21:03:55 +0100

Oliver Tappe <zooey@xxxxxxxxxxxxxxx> wrote:
> >  - if you prefer clear text, the driver_settings come to mind; they 
> > don't have a really nice C++ API (ie. C only), but are used 
> > throughout the 
> > system for various things already (for example, the network 
> > configuration, 
> > see src/server/net/Settings.cpp). Just as a hint to safe some work.
> That is currently being used for the repository config files, as 
> those are 
> easy enough to be parsed as driver_settings. 
> However, .PackageInfo requires multiline strings, too 
> ('description'), so 
> driver_settings wasn't quite fitting. Additionally, I wanted the 
> parser to 
> be as user-friendly as possible (with precise error messages, giving 
> line 
> and column numbers). In the end, writing that parser myself was the 
> only 
> option, maybe that will find it's way to the support kit at a later 
> stage.

That sounds reasonable - it would be nice if the basic structure would 
stay compatible with driver_settings, so that we could use this parser 
for the driver settings API as well, or would that not be a good idea?

Bye,
   Axel.


Other related posts: