all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#6155: Crashes on w32 because of bad frame initialization
@ 2010-05-10 14:13 Lennart Borgman
  2010-05-10 15:12 ` Juanma Barranquero
  0 siblings, 1 reply; 8+ messages in thread
From: Lennart Borgman @ 2010-05-10 14:13 UTC (permalink / raw)
  To: 6155

I have seen numerous crashes on w32 when a frame has been badly
initialized. I wonder if the problem is that frames can get half
initialized.

In the function Fx_create_frame a bad value coming from a frame
parameter (menu-bar-lines . t) will lead to an error in

  x_default_parameter (f, parameters, Qmenu_bar_lines, make_number (1),
		       "menuBar", "MenuBar", RES_TYPE_NUMBER);

Does not that leave the frame half initialized?

(I am loooking at the code from a trunk checkout done 2010-05-06.)





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#6155: Crashes on w32 because of bad frame initialization
  2010-05-10 14:13 bug#6155: Crashes on w32 because of bad frame initialization Lennart Borgman
@ 2010-05-10 15:12 ` Juanma Barranquero
  2010-05-10 16:00   ` Lennart Borgman
  0 siblings, 1 reply; 8+ messages in thread
From: Juanma Barranquero @ 2010-05-10 15:12 UTC (permalink / raw)
  To: Lennart Borgman; +Cc: 6155

> I have seen numerous crashes on w32 when a frame has been badly
> initialized. I wonder if the problem is that frames can get half
> initialized.
>
> In the function Fx_create_frame a bad value coming from a frame
> parameter (menu-bar-lines . t) will lead to an error in
>
>  x_default_parameter (f, parameters, Qmenu_bar_lines, make_number (1),
>                       "menuBar", "MenuBar", RES_TYPE_NUMBER);
>
> Does not that leave the frame half initialized?

Do you have a recipe to reproduce the problem'

    Juanma





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#6155: Crashes on w32 because of bad frame initialization
  2010-05-10 15:12 ` Juanma Barranquero
@ 2010-05-10 16:00   ` Lennart Borgman
  2010-05-10 17:41     ` Juanma Barranquero
  0 siblings, 1 reply; 8+ messages in thread
From: Lennart Borgman @ 2010-05-10 16:00 UTC (permalink / raw)
  To: Juanma Barranquero; +Cc: 6155

On Mon, May 10, 2010 at 5:12 PM, Juanma Barranquero <lekktu@gmail.com> wrote:
>> I have seen numerous crashes on w32 when a frame has been badly
>> initialized. I wonder if the problem is that frames can get half
>> initialized.
>>
>> In the function Fx_create_frame a bad value coming from a frame
>> parameter (menu-bar-lines . t) will lead to an error in
>>
>>  x_default_parameter (f, parameters, Qmenu_bar_lines, make_number (1),
>>                       "menuBar", "MenuBar", RES_TYPE_NUMBER);
>>
>> Does not that leave the frame half initialized?
>
> Do you have a recipe to reproduce the problem'

I have no exact recipe but I noticed the problem when the above
parameter was in the list of parameters to make frame.

The `t' in (menu-bar-lines . t) shoudld be a number. In make-frame I
got an error and shortly afterwards Emacs crashed.





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#6155: Crashes on w32 because of bad frame initialization
  2010-05-10 16:00   ` Lennart Borgman
@ 2010-05-10 17:41     ` Juanma Barranquero
  2010-05-10 18:55       ` Lennart Borgman
  0 siblings, 1 reply; 8+ messages in thread
From: Juanma Barranquero @ 2010-05-10 17:41 UTC (permalink / raw)
  To: Lennart Borgman; +Cc: 6155

On Mon, May 10, 2010 at 18:00, Lennart Borgman
<lennart.borgman@gmail.com> wrote:

> The `t' in (menu-bar-lines . t) shoudld be a number. In make-frame I
> got an error and shortly afterwards Emacs crashed.

(progn
  (push '(menu-bar-lines . t) default-frame-alist)
  (make-frame))

does not fail for me either.

    Juanma





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#6155: Crashes on w32 because of bad frame initialization
  2010-05-10 17:41     ` Juanma Barranquero
@ 2010-05-10 18:55       ` Lennart Borgman
  2010-05-10 19:17         ` Juanma Barranquero
  0 siblings, 1 reply; 8+ messages in thread
From: Lennart Borgman @ 2010-05-10 18:55 UTC (permalink / raw)
  To: Juanma Barranquero; +Cc: 6155

On Mon, May 10, 2010 at 7:41 PM, Juanma Barranquero <lekktu@gmail.com> wrote:
> On Mon, May 10, 2010 at 18:00, Lennart Borgman
> <lennart.borgman@gmail.com> wrote:
>
>> The `t' in (menu-bar-lines . t) shoudld be a number. In make-frame I
>> got an error and shortly afterwards Emacs crashed.
>
> (progn
>  (push '(menu-bar-lines . t) default-frame-alist)
>  (make-frame))

This should perhaps be

  (let ((frame-alist (delete '(menu-bar-lines . 1) default-frame-alist)))
    (setq frame-alist (cons '(menu-bar-lines . t) frame-alist))
    (make-frame frame-alist))

but I do not get an error from that now either. Sigh, I thought I had
tracked down an example that worked. I wonder what is different now.

I will test a bit more.





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#6155: Crashes on w32 because of bad frame initialization
  2010-05-10 18:55       ` Lennart Borgman
@ 2010-05-10 19:17         ` Juanma Barranquero
  2011-07-11 23:16           ` Glenn Morris
  0 siblings, 1 reply; 8+ messages in thread
From: Juanma Barranquero @ 2010-05-10 19:17 UTC (permalink / raw)
  To: Lennart Borgman; +Cc: 6155

> This should perhaps be

No, I don't think it *should*.

>  (let ((frame-alist (delete '(menu-bar-lines . 1) default-frame-alist)))

Unnecessary. `assoc' and friends will find the first matching key.

>    (setq frame-alist (cons '(menu-bar-lines . t) frame-alist))

Identical to my `push', only I was using `default-frame-alist' (on purpose).

>    (make-frame frame-alist))

Unnecessary in my example. `make-frame' already uses `default-frame-alist'.

Please try

(let ((default-frame-alist default-frame-alist))
  (push '(menu-bar-lines . 0) default-frame-alist)
  (make-frame))

to see that it works.

    Juanma





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#6155: Crashes on w32 because of bad frame initialization
  2010-05-10 19:17         ` Juanma Barranquero
@ 2011-07-11 23:16           ` Glenn Morris
  2011-07-12 10:23             ` Lennart Borgman
  0 siblings, 1 reply; 8+ messages in thread
From: Glenn Morris @ 2011-07-11 23:16 UTC (permalink / raw)
  To: 6155-done


In the absence of a recipe to reproduce the issue or a backtrace from a
crash (from a vanilla Emacs with debugging info) I don't think anything
can be done. Please reopen or reply to this report if you get either of
those.





^ permalink raw reply	[flat|nested] 8+ messages in thread

* bug#6155: Crashes on w32 because of bad frame initialization
  2011-07-11 23:16           ` Glenn Morris
@ 2011-07-12 10:23             ` Lennart Borgman
  0 siblings, 0 replies; 8+ messages in thread
From: Lennart Borgman @ 2011-07-12 10:23 UTC (permalink / raw)
  To: 6155, rgm; +Cc: 6155-done

On Tue, Jul 12, 2011 at 01:16, Glenn Morris <rgm@gnu.org> wrote:
>
> In the absence of a recipe to reproduce the issue or a backtrace from a
> crash (from a vanilla Emacs with debugging info) I don't think anything
> can be done. Please reopen or reply to this report if you get either of
> those.

I have not gone this crash any more. Perhaps it was related to one of
the other bug reports about frames. I think I fixed some system call
there, but I do not remember any more.





^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2011-07-12 10:23 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-05-10 14:13 bug#6155: Crashes on w32 because of bad frame initialization Lennart Borgman
2010-05-10 15:12 ` Juanma Barranquero
2010-05-10 16:00   ` Lennart Borgman
2010-05-10 17:41     ` Juanma Barranquero
2010-05-10 18:55       ` Lennart Borgman
2010-05-10 19:17         ` Juanma Barranquero
2011-07-11 23:16           ` Glenn Morris
2011-07-12 10:23             ` Lennart Borgman

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.