James E Keenan wrote:
File src/pmc/classobject.h was added very recently:

r18244 | jonathan | 2007-04-16 19:26:34 -0400 (Mon, 16 Apr 2007) | 1 line
Aye.

I'm wondering why it was added in directory src/pmc/. Previously, that directory held only files with names ending in .num or .pmc:

[parrot] 527 $ ls src/pmc/ | cut -d '.' -f 2 | sort | uniq -c
   1 num
  81 pmc

These were the 'pmc' files transformed by tools/build/pmc2c.pl into c files. Is src/pmc/ the most appropriate location for a .h file?
Only the Class and Object PMCs need this header file. Nothing else should include it. It's part of my attempt to encourage encapsulation. I was following the precedent that src/ contains some .h files (I guess, for a similar reason). I'm pretty sure I've seen .h files show up in languages PMC directories too.

The fact that you've pointed out its unusual for a .h file to be there is perhaps a good thing, 'cus anyone thinking of including it themselves will hopefully think the same. ;-) If anyone feels really strongly otherwise, I won't be bothered if it gets moved.

Jonathan

Reply via email to