<table><tr><td style="">btown created this revision.<br />btown added a reviewer: konsole-devel.<br />btown added a project: Konsole.<br />Restricted Application added a subscriber: Konsole.<br />btown requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D10064" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>From <a href="https://github.com/lxde/qtermwidget/commit/60221da" class="remarkup-link" target="_blank" rel="noreferrer">lxde/qtermwidget@60221da</a> by <a href="https://github.com/yan12125" class="remarkup-link" target="_blank" rel="noreferrer">@yan12125</a>: Support ECMA-48 REP. I tried to make the changes follow existing styles and conventions.</p>

<p><a href="http://www.ecma-international.org/publications/files/ECMA-ST/Ecma-048.pdf" class="remarkup-link" target="_blank" rel="noreferrer">ECMA-48</a> 8.3.103 describes the sequence <strong>CSI Pₙ b</strong> for repeating the previous character in the data stream. This sequence has been present in <a href="http://invisible-island.net/xterm/ctlseqs/ctlseqs.pdf" class="remarkup-link" target="_blank" rel="noreferrer">XTerm</a> since <a href="https://invisible-mirror.net/ncurses/ncurses.faq.html#xterm_generic-id" class="remarkup-link" target="_blank" rel="noreferrer">January 1997</a> and has been <a href="https://invisible-mirror.net/ncurses/terminfo.src.html#t20170729" class="remarkup-link" target="_blank" rel="noreferrer">added to the latest <tt style="background: #ebebeb; font-size: 13px;">terminfo</tt> entry</a> for <tt style="background: #ebebeb; font-size: 13px;">xterm-new</tt> and derived entries such as <tt style="background: #ebebeb; font-size: 13px;">xterm-256color</tt>.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Tested manually with sequences such as:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">x\e[by  <- default is repeat once
x\e[0by  <- 0 means "default", which is repeat once
x\e[4by
 \e[4by  <- space
a™\e[4bz  <- U+2122 TRADE MARK SIGN</pre></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10064" rel="noreferrer">https://phabricator.kde.org/D10064</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/Screen.cpp<br />
src/Screen.h<br />
src/Vt102Emulation.cpp</div></div></div><br /><div><strong>To: </strong>btown, konsole-devel<br /><strong>Cc: </strong>Konsole, ngraham, hindenburg<br /></div>