[PATCH 17/18] mi: miSlideAndSizeWindow -> miResizeWindow

Jasper St. Pierre jstpierre at mecheye.net
Tue Sep 23 12:07:26 PDT 2014


I sort of like "slide" because it gives the impression that we copy the
framebuffer contents if the size of the window hasn't changed. That said,
if we want to change this, we should change the name of the slot too.

Reviewed-by: Jasper St. Pierre <jstpierre at mecheye.net>

On Tue, Sep 23, 2014 at 12:32 PM, Adam Jackson <ajax at redhat.com> wrote:

> Make the function name match the screen slot name.
>
> Signed-off-by: Adam Jackson <ajax at redhat.com>
> ---
>  mi/mi.h        | 12 ++++++------
>  mi/miscrinit.c |  2 +-
>  mi/miwindow.c  |  5 ++---
>  3 files changed, 9 insertions(+), 10 deletions(-)
>
> diff --git a/mi/mi.h b/mi/mi.h
> index 8cb3ce7..5a455c9 100644
> --- a/mi/mi.h
> +++ b/mi/mi.h
> @@ -470,12 +470,12 @@ extern _X_EXPORT void miMoveWindow(WindowPtr /*pWin
> */ ,
>                                     VTKind       /*kind */
>      );
>
> -extern _X_EXPORT void miSlideAndSizeWindow(WindowPtr /*pWin */ ,
> -                                           int /*x */ ,
> -                                           int /*y */ ,
> -                                           unsigned int /*w */ ,
> -                                           unsigned int /*h */ ,
> -                                           WindowPtr    /*pSib */
> +extern _X_EXPORT void miResizeWindow(WindowPtr /*pWin */ ,
> +                                     int /*x */ ,
> +                                     int /*y */ ,
> +                                     unsigned int /*w */ ,
> +                                     unsigned int /*h */ ,
> +                                     WindowPtr    /*pSib */
>      );
>
>  extern _X_EXPORT WindowPtr miGetLayerWindow(WindowPtr   /*pWin */
> diff --git a/mi/miscrinit.c b/mi/miscrinit.c
> index ec4d108..327bd15 100644
> --- a/mi/miscrinit.c
> +++ b/mi/miscrinit.c
> @@ -264,7 +264,7 @@ miScreenInit(ScreenPtr pScreen, void *pbits,  /*
> pointer to screen bits */
>      pScreen->MarkWindow = miMarkWindow;
>      pScreen->MarkOverlappedWindows = miMarkOverlappedWindows;
>      pScreen->MoveWindow = miMoveWindow;
> -    pScreen->ResizeWindow = miSlideAndSizeWindow;
> +    pScreen->ResizeWindow = miResizeWindow;
>      pScreen->GetLayerWindow = miGetLayerWindow;
>      pScreen->HandleExposures = miHandleValidateExposures;
>      pScreen->ReparentWindow = (ReparentWindowProcPtr) 0;
> diff --git a/mi/miwindow.c b/mi/miwindow.c
> index ba4c9ed..cfe085a 100644
> --- a/mi/miwindow.c
> +++ b/mi/miwindow.c
> @@ -337,9 +337,8 @@ miRecomputeExposures(WindowPtr pWin, void *value)
>  }
>
>  void
> -miSlideAndSizeWindow(WindowPtr pWin,
> -                     int x, int y,
> -                     unsigned int w, unsigned int h, WindowPtr pSib)
> +miResizeWindow(WindowPtr pWin, int x, int y, unsigned int w, unsigned int
> h,
> +               WindowPtr pSib)
>  {
>      WindowPtr pParent;
>      Bool WasViewable = (Bool) (pWin->viewable);
> --
> 1.9.3
>
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
>



-- 
  Jasper
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.x.org/archives/xorg-devel/attachments/20140923/08a66f9b/attachment.html>


More information about the xorg-devel mailing list