comctl32_tests: Janitorial: write-strings warning fix

Robert Shearman rob at codeweavers.com
Sat May 27 17:29:55 CDT 2006


Andrew Talbot wrote:

>Because I have had at least one patch like this accepted, I assumed what I
>was doing was OK. I guess another way of preventing this warning would be
>to use:
>
>    x.pszText = (LPSTR) "string";
>
>Assuming there is no danger in using this method, then it is neater for a
>single assignment. The array method may be better when the same text is
>assigned several times in the same function or file.
>  
>

This method should be safe for this particular instance.

>If what I am doing is incorrect or not helpful, I would welcome advice to
>that effect. After all, I'm trying to make myself useful, not mess things
>up.
>

Absolutely. We should avoid regressions at all costs, but making literal 
strings non-writable should help to uncover bugs.

-- 
Rob Shearman




More information about the wine-devel mailing list