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

  • From: "Adrien Destugues" <pulkomandy@xxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Wed, 21 Nov 2018 15:48:07 +0000

21 novembre 2018 15:40 "Dario Casalinuovo" <b.vitruvio@xxxxxxxxx> a écrit:

On Wed, Nov 21, 2018 at 3:03 PM Adrien Destugues <pulkomandy@xxxxxxxxxxxxx> 
wrote:

+#define P BPrivate::media::

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

Also I would consider #define P BPrivate::media and then use P::Stuff for 
better readability and
ease of grep'ing.

Just curious why does that help grepping?

I will find myself looking for PStuff (in sources, or in readelf or other 
symbol listings) when I should be looking for BPrivate::media::Stuff. When 
P::Stuff is used I will more easily remember that P is an abbreviated name for 
a namespace,
and not directly part of the class name.

-- 
Adrien.


Other related posts: