[PATCH 2/2] riched20: Use ME_GetSelectionType() in ME_ShowContextMenu() for getting type of selection.

Huw Davies huw at codeweavers.com
Thu Apr 19 02:28:07 CDT 2018


On Tue, Apr 17, 2018 at 10:49:01PM +0800, Jactry Zeng wrote:
> Signed-off-by: Jactry Zeng <jzeng at codeweavers.com>
> ---
>  dlls/riched20/editor.c | 13 +++----------
>  1 file changed, 3 insertions(+), 10 deletions(-)
> 
> 

> diff --git a/dlls/riched20/editor.c b/dlls/riched20/editor.c
> index df1f1870cd..74540f88d9 100644
> --- a/dlls/riched20/editor.c
> +++ b/dlls/riched20/editor.c
> @@ -3005,19 +3005,12 @@ static BOOL ME_ShowContextMenu(ME_TextEditor *editor, int x, int y)
>  {
>    CHARRANGE selrange;
>    HMENU menu;
> -  int seltype = 0;
> +  int seltype = SEL_EMPTY;
> +
>    if(!editor->lpOleCallback || !editor->hWnd)
>      return FALSE;
>    ME_GetSelectionOfs(editor, &selrange.cpMin, &selrange.cpMax);
> -  if(selrange.cpMin == selrange.cpMax)
> -    seltype |= SEL_EMPTY;
> -  else
> -  {
> -    /* FIXME: Handle objects */
> -    seltype |= SEL_TEXT;
> -    if(selrange.cpMax-selrange.cpMin > 1)
> -      seltype |= SEL_MULTICHAR;
> -  }
> +  ME_GetSelectionType(editor);

You'll want to actually use the selection type ;-/

>    if(SUCCEEDED(IRichEditOleCallback_GetContextMenu(editor->lpOleCallback, seltype, NULL, &selrange, &menu)))
>    {
>      TrackPopupMenu(menu, TPM_LEFTALIGN | TPM_RIGHTBUTTON, x, y, 0, editor->hwndParent, NULL);
> 





More information about the wine-devel mailing list