Some of this was included in the patch submission.&nbsp; Some of it I had questions about.<br><br>spawn.c : warnings may or may not be harmless depending on if esecvp attempts to modify the results.&nbsp; Specs indicate it doesn&#39;t, but the function typing doesn&#39;t indicate so(the difference is &#39;const char * const argv[]&#39; and &#39;char *const argv[]&#39;)

libs/wine/string.c: &nbsp;&nbsp; &nbsp;strstrW - returns the location where sub was found in str.&nbsp; I believe returning a non-const value is what is expected.&nbsp; Should this be changed to a non-const parameter, or just leave the warning as is?

<br><br>tools/winedump/pdb.c:<br>

&nbsp;&nbsp; &nbsp;pdb_exit<br>

&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;[line 149,153,155] - freeing of const variable.&nbsp; Is this ok?&nbsp; I could imagine it *could* cause problems if what we are freeing isn&#39;t in readable/writeable memory.<br>

&nbsp;&nbsp; &nbsp;pdb_dump_symbols<br>

&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;[line 293,403,409] - same as pdb_exit.<br>

&nbsp;&nbsp; &nbsp;pdb_dump_types<br>

&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;[line 462] - ditto<br>

&nbsp;&nbsp; &nbsp;pdb_jg_dump<br>

&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;line[514] - ditto<br>

&nbsp;&nbsp; &nbsp;pdb_ds_dump<br>

&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;line[643] - ditto<br>

<br>tools/winedump/search.c:<br>

&nbsp;&nbsp; &nbsp;get_type<br>

&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;[line 305, 329] - freeing of const variable again.<br>

&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;[line 314, 335] - both cases, set return_text to type_str.&nbsp;

Either parsed_symbol&#39;s return_text needs to be constified, proto should

be unconstified, or leave it in place as a necessary cast.<br>

Any comments on the above spots?&nbsp; what of these are necessary casts?&nbsp; what of these can/should be corrected, and what needs to be done to correct them? **patches submitted to wine-patches** the following were straight forward patches that shouldn&#39;t modify the flow of the program beyond fixing the warnings.

libs/wine/string.c: &nbsp;&nbsp; &nbsp;strtolW and strtoulW - constify the endptr value.&nbsp; This is just an output value to indicate where in the array we stoped processing.&nbsp; Change causes this to be the same type as the first value.&nbsp; Reduces number of cast-qual warnings by 6.

tools/winedump/lnk.c: &nbsp;&nbsp; &nbsp;dump_pidl &nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;[line 176] - switched to const.&nbsp; Value is never modified in the function.&nbsp; Removes a cast-qual warning. tools/winedump/symbol.c: &nbsp;&nbsp; &nbsp;symbol_clean_string - this function is modifying the string.&nbsp; This should most definitely *not* be a const parameter.&nbsp; It&#39;s not even logically const(as a comment tried to claim) as the the intent of the function is to make the type string Wine-friendly.&nbsp; That requires modifications to that string, and to the variable.&nbsp; Removes a cast-qual warning.

Comments? questions? Current number of cast-qual warnings fixed: 8. Number deemed necessary: 0(you guys need to help me decide what of these need to remain) Number looked at: 24. Estimated number remaining: 521.