Hirohito Higashi wrote:

[...]

> > > BTW...
> > > > > But when we get ESC]11; and then no BEL or ESC\ follows, we never get
> > > > > out of this loop, we run into the "not enough characters" part 
> > > > > forever.
> > > 
> > > This problem is also in the process of the current of CSI and DCS.
> > > Find and check for below string in term.c
> > >   "Not enough characters for CRV"
> > >   "not enough characters for XT"
> > 
> > We might need to find a generic solution.  A timeout is probably
> > appropriate, these escape sequences should be completely received within
> > a short time.  Although on a slow connection it might get split (the
> > "entering a tunnel" problem).
> 
> Sure.
> This matter is corresponding later.
> 
> I updated a patch. (BELL termination check were missing for RBG)
> 
> And I received a review from Hayaki Saito.
> He says:
> - 8-bit response can not received. (Contains mouse input)
> - may_req_bg_color() call timing is too early. Should Called after t_CRV 
> response received and switch to 8-bit mode.
> 
> The findings, I would try to correspond to the weekend.
> 
> These are the issues from before.
> So, it might be good to include rbg_fix5.patch first :-)

OK, let's first make it a bit more consistent.  I think it's good to
check the first characters after DCS and OSC to avoid eating an escape
sequence that someone might map.  Hopefully we don't let anything
through that causes problems, we'll have to see if someone complains.

This also reduces the chance of looping to get more characters for
anything that starts with DCS or OSC that ends in another way than we
expect.  So perhaps the timeout isn't needed.

-- 
Some of the well known MS-Windows errors:
        EHUH            Unexpected error
        EUSER           User error, not our fault!
        EGOD            Horrible problem, god knows what has happened
        EERR            Errornous error: nothing wrong

 /// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\  an exciting new programming language -- http://www.Zimbu.org        ///
 \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

-- 
-- 
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].
For more options, visit https://groups.google.com/d/optout.

Raspunde prin e-mail lui