bug#18215: Fwd: Re: bug#18215: 24.4.50; OSX 10.6.8; set-frame-size by pixelwise does not work following `make-fame`.

[Keith David Bershatsky <esq@lawlist.com>]

[-- Attachment #1: Type: text/plain, Size: 195 bytes --]

The following is a copy of a message that should have been sent to `18215@debbugs.gnu.org`, but which was inadvertently omitted.

;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;


[-- Attachment #2: Type: message/rfc822, Size: 1995 bytes --]

From: Keith David Bershatsky <esq@lawlist.com>
To: martin rudalics <rudalics@gmx.at>
Subject: Re: bug#18215: 24.4.50; OSX 10.6.8; set-frame-size by pixelwise does not work following `make-fame`.
Date: Sat, 09 Aug 2014 09:39:50 -0700
Message-ID: <m2ha1l4nxl.wl%esq@lawlist.com>

Martin:

The patch you provided did indeed fix the problem -- both examples mentioned in the initial bug report now work with the patched build of Emacs Trunk from August 8, 2014.

I should mention that there is a slight difference between the June 1, 2014 build and the patched build of August 8, 2014.  The difference is about six (6) pixels in frame width.  With the June 1, 2014 build, I use 1894 to exactly fill the screen that is 1920 pixels wide.  With the patched build of August 8, 2014, I would now need to use 1900 to exactly fill that same screen which is 1920 pixels wide.  `(frame-pixel-width)` returns 1920 with the patched build following (set-frame-size (selected-frame) 1900 1054 t).

Thanks,

Keith

;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;

At Sat, 09 Aug 2014 11:44:02 +0200,
martin rudalics wrote:
> 
>  > The version of Emacs Trunk built on June 1, 2014 works well -- i.e., the two examples in the current bug report work as expected.
>  >
>  > GNU Emacs 24.4.50.1 (x86_64-apple-darwin10.8.0, NS appkit-1038.36 Version 10.6.8 (Build 10K549)) of 2014-06-01 on MP.local
>  >
>  > I did not test any builds between the dates of June 2, 2014 through August 6, 2014.
>  >
>  > Yes, `(set-frame-size (selected-frame) 1894 1054 t)`, works well with the August 7 and 8, 2014 build when it is evaluated all by itself.
>  >
>  > `M-x toggle-frame-maximized` works well with the August 7 and 8, 2014 build.
> 
> Thanks for the information.  Can you please try the attached patch?
> 
> Thanks, martin