
Comments/corrections for the 'Extending Python' article

(1) I have obtained permission to use the email
 'greg@siliconoptix.com' in this article.

(2) Regarding dropping of three of the six proposed listings:
please consider re-introducing Listing 4. The article discusses
four examples, two of which are 'usual practice', and two illustrate
the desired technique. Listing 4, which is only 23 lines, is important
to complete the second example. It would be very nice to have Listing
5 as well, but of course it is 83 lines. The text discusses how
stats3module.c is composed from elements of statsimodule and stats2.py;
so including Listing 4 (stats2.py) will also serve to illustrate the
third example to some extent, even if Listing 5 cannot be used.

(3) Change second Pullquote to

"Rather than putting the wrapper
function in a separate Python file, you can include it in the C module."

There is a corresponding change in the text.

(4) Corrections for the text
===================================================================
Line 2.37:
change to " ... spend a lot of time ..."

===================================================================
Line 2.120:
a bit clumsy,
          " ... needs to look for a lot of error cases, even ..."
change to " ... needs to check for several error conditions, even..."

===================================================================
Line 3.19:
Word built-in should not be in italics, should be hyphenated 

===================================================================
Line 3.22:
Word 'built-in' should not be in italics, should be hyphenated 
and in quotes.

===================================================================
Line 3.25:
Should have single quotes around letter 'f', this a fragment of
source code.

===================================================================
Line 3.30:
Word built-in should not be in italics, should be hyphenated 

===================================================================
Line 3.42:
Should say "import stats2", not "import stats".

===================================================================
Line 3.65:
Should be lowercase 'to' after colon

===================================================================
Line 3.88:
Change sentence to: "Version incompatibility problems can be
avoided by putting the C code and the Python code in the
same file".

===================================================================
Line 3.105:
Clumsy sentence. Change to "So, rather than putting the wrapper
function in a separate Python file, you can include it in the C module."

===================================================================
Line 4.81:
Should be "... the Python layer. Just change this:"

===================================================================
Line 4.123:
change "parameter sets." to "parameter set."

===================================================================
Line 5.18:
Over-long sentence:
change to " ... raise exceptions. The drawback is ..."

===================================================================
Line 5.37:
Intent is unclear, change to 
"... fault, I want to be sure that this did not occur simply ..."

===================================================================


(5) Listing 3 has a couple of typos, I've attached a corrected version.
Please also replace 'statsimodule.c' in the downlodable files.

	if( n < 0
should be
	if( n <= 0
and there is a spelling error 'lengtht' a few lines down.

============ END ====================================



