Richedit: fix bug 14280, EM_SETTEXTTEX obeys ST_SELECTION with RTF inputs

Austin English austinenglish at gmail.com
Mon Jul 28 06:09:35 CDT 2008


On Sun, Jul 27, 2008 at 9:52 PM, Dustin Brody <ne4 at parsoma.net> wrote:
> I sent the patch at
> http://www.winehq.org/pipermail/wine-patches/2008-July/058460.html but
> got neither it applied nor any other comment. As far as I can tell, I
> complied with all the guidelines at
> http://www.winehq.org/site/sending_patches.
>
> Is it the editing of comments not strictly part of the new test (but,
> I'd claim, valuable for the future expansion of those test cases to
> cover the remaining flag option combinations)? The introduction of a new
> buffer in the test handler? That the test handler maybe should be a
> patch in itself (although the standard practise seems to be to bundle
> them)? The code in the new test-cases? That the editor.c patch should
> adopt a different approach? That aside from my own patch to DC++
> (discussed in the numbered bug report) I haven't looked for or found a
> real-world program using this (in which case, should I first get the
> DC++ patch in, then point to that as an example of such a real-world
> program; I have been avoiding that precisely because I primarily use
> WINE and don't want to break things for myself in the indeterminately
> long period after such an application to the main DC++ tree)? Something
> else?
>
> Aside from the test cases, this is literally a one-bit change from 0 to
> 1 and it's been a weeks and a resend and I have no idea what the status
> is, how likely it is ever to be committed, or how I might have screwed
> up. Feedback would be appreciated.
>
> -Dustin Brody
>
>
>

Splitting the testcase from the fix would be helpful.



More information about the wine-devel mailing list