Message 99833 - Python tracker

changing the definition to (const char *) seems like the right thing to do - a quick grep of the Python source and a search on google codesearch only shows uses with either string literals or string literals cast to (char *) in order to silence a warning.  I tried changing it on the 2.x trunk and it compiles with no warnings.

+1, any other opinions?