[haiku-commits] Re: r42668 - in haiku/trunk: headers/private/graphics/common src/add-ons/accelerants/common

  • From: "Axel Dörfler" <axeld@xxxxxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Mon, 22 Aug 2011 10:37:01 +0200 (MEST)

kallisti5@xxxxxxxxxxx wrote:
>    haiku/trunk/headers/private/graphics/common/video_electronics.h

Maybe video_configuration.h would be a better name?

> * add video_electronics.c
>   (.c to keep compatibility with older C accelerants)

No need. As long as you export the names C-only (ie. with extern "C"), the 
sources can still be C++.

> +// mostly for debugging detected monitors
> +const char* decode_connector_name(uint32 connector);
> +const char* decode_encoder_name(uint32 encoder);

'decode...name' doesn't really fit. Why not just 'get' instead?

Bye,
   Axel.


Other related posts: