<div dir="auto">On the one hand dropping redundancy in the filename is fine.  On the other having the names mirror the public header files is valuable.  How about leaving the base names alone and change the directory to "pyinternal"?<div dir="auto"><br></div><div dir="auto">-eric</div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Oct 31, 2018, 17:36 Victor Stinner <<a href="mailto:vstinner@redhat.com">vstinner@redhat.com</a> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ok, thanks. I decided to remove the redundant "py", so I renamed<br>
"pystate.h" to "pycore_state.h" (single "py", instead of<br>
"pycore_pystate.h", double "py py").<br>
<br>
I updated my PR:<br>
<a href="https://github.com/python/cpython/pull/10263" rel="noreferrer noreferrer" target="_blank">https://github.com/python/cpython/pull/10263</a><br>
<br>
* Rename Include/internal/ header files:<br>
<br>
  * pyatomic.h -> pycore_atomic.h<br>
  * ceval.h -> pycore_ceval.h<br>
  * condvar.h -> pycore_condvar.h<br>
  * context.h -> pycore_context.h<br>
  * pygetopt.h -> pycore_getopt.h<br>
  * gil.h -> pycore_gil.h<br>
  * hamt.h -> pycore_hamt.h<br>
  * hash.h -> pycore_hash.h<br>
  * mem.h -> pycore_mem.h<br>
  * pystate.h -> pycore_state.h<br>
  * warnings.h -> pycore_warnings.h<br>
<br>
* PCbuild project, <a href="http://Makefile.pre.in" rel="noreferrer noreferrer" target="_blank">Makefile.pre.in</a>, Modules/Setup: add the<br>
  Include/internal/ directory to the search paths of header files.<br>
* Update include. For example, replace #include "internal/mem.h"<br>
  with #include "pycore_mem.h".<br>
<br>
Victor<br>
Le mer. 31 oct. 2018 à 23:38, Eric Snow <<a href="mailto:ericsnowcurrently@gmail.com" target="_blank" rel="noreferrer">ericsnowcurrently@gmail.com</a>> a écrit :<br>
><br>
> B<br>
> On Wed, Oct 31, 2018 at 4:28 PM Victor Stinner <<a href="mailto:vstinner@redhat.com" target="_blank" rel="noreferrer">vstinner@redhat.com</a>> wrote:<br>
> ><br>
> > Le mer. 31 oct. 2018 à 22:19, Eric Snow <<a href="mailto:ericsnowcurrently@gmail.com" target="_blank" rel="noreferrer">ericsnowcurrently@gmail.com</a>> a écrit :<br>
> > > > Maybe we can keep "Include/internal/" directory name, but add<br>
> > > > "pycore_" rather than "internal_" to header filenames?<br>
> > ><br>
> > > this sounds good to me.  thanks for chasing this down.<br>
> ><br>
> > What do you prefer?<br>
> ><br>
> > (A Include/internal/internal_pystate.h<br>
> > (B) Include/internal/pycore_pystate.h<br>
> > (C) Include/pycore/pycore_pystate.h<br>
> ><br>
> > Victor<br>
</blockquote></div>