On Di, 08 Dez 2020, Christian Brabandt wrote:

> > Am 08.12.2020 um 21:13 schrieb Bram Moolenaar <[email protected]>:
> > Patch 8.2.2113
> > Problem:    MS-Windows GUI: crash after using ":set guifont=" four times.
> > Solution:   Check for NULL pointer. (Ken Takata, closes #7434)
> > Files:      src/gui_dwrite.cpp, src/testdir/test_gui.vim
> 
> Thanks!
> I think the test additionally needs a :set renderoptions=type:directx
> or perhaps should test with both setting with and without renderoptions.

The current test does not make gvim crash. How about to change the test like 
this:

for item in ['type:directx', '']
  exe ':set renderoptions='.item
  set guifont=Consolas:h9
  " doing this four times used to cause a crash
  for i in range(5)
      set guifont=
  endfor
  " force a redraw
  redraw!
endfor

Unfortunately, this still does not make my gvim crash. So apparently, 
something else is needed to make `:set guifn=` make gvim crash.

(I already tried adding `call test_override('char_avail', 1)`, around 
it, but this did not change anything). But it might still make sense to 
test setting guifont with and withouth setting renderoptions.


Best,
Christian
-- 
Jetzt wächst zusammen, was zusammen gehört.
                -- Willy Brandt

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/vim_dev/20201209072541.GI22416%40256bit.org.

Raspunde prin e-mail lui