[haiku-commits] Re: haiku: hrev52569 - src/kits/codec headers/os/codec

  • From: Dario Casalinuovo <b.vitruvio@xxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Wed, 21 Nov 2018 15:28:09 +0100

On Wed, Nov 21, 2018 at 2:56 PM Axel Dörfler <axeld@xxxxxxxxxxxxxxxx> wrote:

Am 21/11/2018 um 13:02 schrieb Barrett17:
   * Use string keys. I am still convinced we need BValue.

You mean something like this:
https://git.haiku-os.org/haiku/tree/headers/private/shared/Variant.h
?

Since BMetaData is now practically a BMessage, do you need this class at
all?
What additional use does it have?


Yes as I told you in an earlier message, the aim is to enforce a certain
use of BMessage.
Using BMessage is too fragile in this context. A BMetaData field is the
only one allowed to have multiple occurrences.
Other fields should be single fields.


+#define P BPrivate::media::

Maybe I just missed it, but I didn't see a "#undef P".


Right.

-- 
Saluti,
Dario

Other related posts: