[PATCH v3 2/2] riched20: Only pass valid length to ME_InsertTextFromCursor
Fabian Maurer
dark.shadow4 at web.de
Tue Jan 10 12:14:07 CST 2017
On Tuesday, January 10, 2017 10:07:48 AM CET Huw Davies wrote:
> I'm not convinced this is an improvement.
I figured it would be nice for uniformity. I find it cleaner to always calculate
the size before - or in the function. Currently we have only 2 cases where the
size is calculated inside, in all other cases it's passed.
As it is now, you can pass the function a null pointer, but only if the length
is 0. If length is >0, or -1, it crashes.
After my patch, the function requires the length being set properly, but
always works with a nullpointer.
Fabian
More information about the wine-devel
mailing list