<br><br><div class="gmail_quote">On Mon, Jun 9, 2008 at 1:44 AM, M.-A. Lemburg &lt;<a href="mailto:mal@egenix.com">mal@egenix.com</a>&gt; wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">On 2008-06-09 07:20, Gregory P. Smith wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
On Fri, Jun 6, 2008 at 2:19 AM, M.-A. Lemburg &lt;<a href="mailto:mal@egenix.com" target="_blank">mal@egenix.com</a>&gt; wrote:<br>
<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
On 2008-06-03 01:29, Gregory P. Smith wrote:<br>
<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
On Mon, Jun 2, 2008 at 4:09 PM, Guido van Rossum &lt;<a href="mailto:guido@python.org" target="_blank">guido@python.org</a>&gt;<br>
wrote:<br>
<br>
&nbsp;I will freely admit that I haven&#39;t followed this thread in any detail,<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
but if it were up to me, I&#39;d have the 2.6 internal code use PyString<br>
<br>
</blockquote>
...<br>
<br>
Should we read this as a BDFL pronouncement and make it so?<br>
<br>
All that would mean change wise is that trunk r63675 as well as possibly<br>
r63672 and r63677 would need to be rolled back and this whole discussion<br>
over if such a big change should have happened would turn into a moot<br>
point.<br>
<br>
</blockquote>
I would certainly welcome reverting the change.<br>
<br>
All that&#39;s needed to support PyBytes API in 2.x is a set of #defines<br>
that map the new APIs to the PyString names. That&#39;s a clean and<br>
easily understandable solution.<br>
<br>
</blockquote>
<br>
Okay, I&#39;ve reverted r63675 in trunk revision r64048. &nbsp;That leaves all of the<br>
python modules and internals using PyString_ api names instead of PyBytes_<br>
api names as they were before. &nbsp;PyBytes_ #define&#39;s exist for the appropriate<br>
PyString methods incase anyone wants to use those.<br>
</blockquote>
<br></div>
Thanks.<div class="Ih2E3d"><br>
<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Programmers interested in the code<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
for a PyString API can then still look up the code in stringobject.c,<br>
e.g. to find out how a certain special case is handled or to check<br>
the ref counting - just like they did for years.<br>
</blockquote>
<br>
<br>
The files still exist with the new names. &nbsp;bytesobject.c instead of<br>
stringobject.c. &nbsp;Those renames were done in the other CLs i mentioned which<br>
have not yet been reverted. &nbsp;The current state seems a bit odd because they<br>
depend on the #defines to cause method definitions to be the PyString_ names<br>
instead of the PyBytes_ names.<br>
</blockquote>
<br></div>
Please restore the original state, ie. PyString APIs live in<br>
stringobject.h and stringobject.c. bytesobject.h should then have<br>
the #defines for PyBytes APIs, pointing them to the PyString<br>
names (basically what&#39;s currently in stringobject.h).<div class="Ih2E3d"><br>
</div></blockquote><div><br>all done as of 64105<br>&nbsp;<br></div></div><br>