RFC: Richedit: Test Patch for EM_SETMARGINS
James McKenzie
jjmckenzie51 at earthlink.net
Sun Jan 16 11:50:55 CST 2011
On 1/16/11 10:24 AM, André Hentschel wrote:
>
> see http://test.winehq.org/data/
>
Will do. I don't want to break Windows9x functionality though.
>> + /* Per http://msdn.microsoft.com/en-us/library/bb761649%(VS.85).asp
>> if the lparam is
>>>> + EC_USEFONTINFO the Left and Right Margins should be set to a narrow width if the
>>>> + font has been set */
>>> They don't use permanent links I believe.
>> Can I modify this to state the page name?
> remove the url, maybe better something like "msdn's documentation for functionxyz..."
>
Much better idea.
> + todo_wine {
>>>> + SendMessageW(hwndRichEdit, EM_SETMARGINS, EC_USEFONTINFO, MAKELONG(EC_USEFONTINFO, EC_USEFONTINFO));
>>>> + }
>>> Hm.
>> This is one of the test cases that was recommended and it appears to be the only one that 'works'.
> i bet Nikolay sighed about the curly brackets
>
Same style as used in the rest of the file. I like curly braces on
separate lines (I hope that is what you are referring to.)
>>>> + if (winetest_debug> 1) {
>>>> test_WM_CHAR();
>>>> test_EM_FINDTEXT();
>>>> test_EM_GETLINE();
>>>> @@ -7126,6 +7643,8 @@ START_TEST( editor )
>>>> test_WM_GETDLGCODE();
>>>> test_zoom();
>>>> test_dialogmode();
>>>> + }
>>>> + test_em_setmargins();
>>> Why?
>>>
>>> Another question is how screen resolution affects that test.
>> I will look at this as well. I'll have to figure out how to change the resolution on the test bot and or my local copy of WindowsXP SP2 (not upgradable, no Internet connection to that system.
> to be clear: i guess it's about the dpi and not the resolution like 1024x768.
>
I interpreted it that way as well. I have a system here set to 120 DPI
and use small fonts for the default font setting.
James McKenzie
More information about the wine-devel
mailing list