[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: