Re: [vim/vim] patch 9.0.0815 (9c50eeb)

34 views
Skip to first unread message

Christian Brabandt

unread,
Oct 22, 2022, 4:25:54 AM10/22/22
to vim/vim, Subscribed

Hi Bram,
now we are missing description of the patch and we still do not have a tag :(


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/commit/9c50eeb40117413bf59a9da904c8d0921ed0a6e6/87568768@github.com>

Björn Steinbrink

unread,
Oct 22, 2022, 4:58:14 AM10/22/22
to vim/vim, Subscribed

This causes an immediate use-after-free, doesn't it?


Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/commit/9c50eeb40117413bf59a9da904c8d0921ed0a6e6/87570042@github.com>

Björn Steinbrink

unread,
Oct 22, 2022, 5:06:33 AM10/22/22
to vim/vim, Subscribed

Yeah, reverting this fixing both the segfault and the broken :! behaviour I mentioned in #11409 (comment)
And from 9.0.0783 and 9.0.0785 this doesn't seem necessary. What am I missing?


Reply to this email directly, view it on GitHub.

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/commit/9c50eeb40117413bf59a9da904c8d0921ed0a6e6/87570425@github.com>

zeertzjq

unread,
Oct 22, 2022, 5:16:38 AM10/22/22
to vim/vim, Subscribed

It seems that there was a memory leak in the PR.


Reply to this email directly, view it on GitHub.

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/commit/9c50eeb40117413bf59a9da904c8d0921ed0a6e6/87570875@github.com>

Björn Steinbrink

unread,
Oct 22, 2022, 7:10:22 AM10/22/22
to vim/vim, Subscribed

Oh, you mean #11420? Yeah, taking a closer look, that seems to be the case, thanks!


Reply to this email directly, view it on GitHub.

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/commit/9c50eeb40117413bf59a9da904c8d0921ed0a6e6/87575737@github.com>

Björn Steinbrink

unread,
Oct 22, 2022, 7:12:35 AM10/22/22
to vim/vim, Subscribed

AFAICT this will cause :! to redo the previous command if shellquote is set, because the quoting code below uses prevcmd


Reply to this email directly, view it on GitHub.

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/commit/9c50eeb40117413bf59a9da904c8d0921ed0a6e6/87575841@github.com>

Bram Moolenaar

unread,
Oct 27, 2022, 2:46:40 PM10/27/22
to vim/vim, Subscribed


Christian wrote:

> now we are missing description of the patch and we still do not have a tag :(

How do I fix that?

--
If you don't get everything you want, think of
everything you didn't get and don't want.

/// Bram Moolenaar -- ***@***.*** -- http://www.Moolenaar.net \\\
/// \\\
\\\ sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/commit/9c50eeb40117413bf59a9da904c8d0921ed0a6e6/88108037@github.com>

Bram Moolenaar

unread,
Oct 28, 2022, 5:24:10 PM10/28/22
to vim/vim, Subscribed


I thought I had fixed the problems, but now I see the filetype test is
failing. Can someone see why? It's like the Mac file just disappears.

--
Eye have a spelling checker, it came with my PC;
It plainly marks four my revue mistakes I cannot sea.
I've run this poem threw it, I'm sure your please to no,
It's letter perfect in it's weigh, my checker tolled me sew!


/// Bram Moolenaar -- ***@***.*** -- http://www.Moolenaar.net \\\
/// \\\
\\\ sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///


Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/commit/9c50eeb40117413bf59a9da904c8d0921ed0a6e6/88233761@github.com>

Bram Moolenaar

unread,
Oct 28, 2022, 5:47:24 PM10/28/22
to vim...@googlegroups.com, reply+ACY5DGDBWY34XN5T4F...@reply.github.com
I mean the fileformat test, filetype looks OK.  Probably related to the 'endoffile' option change.
I'm too tired right now to look into it. Helps a lot if someone can provide a PR.

--
--
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 vim_dev+u...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/vim_dev/vim/vim/commit/9c50eeb40117413bf59a9da904c8d0921ed0a6e6/88233761%40github.com.

vim-dev ML

unread,
Oct 28, 2022, 5:47:44 PM10/28/22
to vim/vim, vim-dev ML, Your activity

I mean the fileformat test, filetype looks OK. Probably related to the
'endoffile' option change.
I'm too tired right now to look into it. Helps a lot if someone can provide
a PR.

On Fri, 28 Oct 2022 at 22:24, Bram Moolenaar ***@***.***>

wrote:

>
> I thought I had fixed the problems, but now I see the filetype test is
> failing. Can someone see why? It's like the Mac file just disappears.
>
> --
> Eye have a spelling checker, it came with my PC;
> It plainly marks four my revue mistakes I cannot sea.
> I've run this poem threw it, I'm sure your please to no,
> It's letter perfect in it's weigh, my checker tolled me sew!
>
> /// Bram Moolenaar -- ***@***.*** -- http://www.Moolenaar.net \\\
> /// \\\
> \\\ sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
> \\\ help me help AIDS victims -- http://ICCF-Holland.org ///
>
> —
> Reply to this email directly, view it on GitHub
> <https://github.com/vim/vim/commit/9c50eeb40117413bf59a9da904c8d0921ed0a6e6#r88233761>
> .
> You are receiving this because you are subscribed to this thread.Message
> ID: <vim/vim/commit/9c50eeb40117413bf59a9da904c8d0921ed0a6e6/88233761@

> github.com>
>
> --
> --
> 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 ***@***.*** > <https://groups.google.com/d/msgid/vim_dev/vim/vim/commit/9c50eeb40117413bf59a9da904c8d0921ed0a6e6/88233761%40github.com?utm_medium=email&utm_source=footer>
> .
>


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/commit/9c50eeb40117413bf59a9da904c8d0921ed0a6e6/88235318@github.com>

Anton Sharonov (ant0sha)

unread,
Oct 29, 2022, 4:29:00 PM10/29/22
to vim/vim, vim-dev ML, Comment

I thought I had fixed the problems, but now I see the filetype test is failing. Can someone see why? It's like the Mac file just disappears.

I mean the fileformat test, filetype looks OK. Probably related to the 'endoffile' option change.

Seems to be fixed with 9.0.0819


Reply to this email directly, view it on GitHub, or unsubscribe.

You are receiving this because you commented.Message ID: <vim/vim/commit/9c50eeb40117413bf59a9da904c8d0921ed0a6e6/88297249@github.com>

Reply all
Reply to author
Forward
0 new messages