Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

readline,repl: add substring history search #31112

Closed
wants to merge 14 commits into from
Closed

readline,repl: add substring history search #31112

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
05c19bd
repl: improve preview length calculation
BridgeAR Dec 26, 2019
470e87f
readline,repl: add substring based history search
BridgeAR Dec 27, 2019
6efc2b9
readline,repl: skip history entries identical to the current line
BridgeAR Dec 27, 2019
5a213fe
readline,repl: improve history up/previous
BridgeAR Dec 29, 2019
3b8315c
readline: simplify getStringWidth()
BridgeAR Dec 31, 2019
d6b1bac
readline: move charLengthLeft() and charLengthAt()
BridgeAR Dec 31, 2019
9f7730e
src: improve GetColumnWidth performance
BridgeAR Dec 31, 2019
e9bc956
src: change GetStringWidth's expand_emoji_sequence option default
BridgeAR Dec 31, 2019
6dd223b
readline,repl: support tabs properly
BridgeAR Dec 31, 2019
26d4643
repl: activate previews for lines exceeding the terminal columns
BridgeAR Jan 4, 2020
387a327
test: add repl tests to verify unicode support in previews
BridgeAR Jan 4, 2020
9f5aa54
util: add todo comments for inspect to add unicode support
BridgeAR Jan 4, 2020
f278c06
squash: readline: simplify getStringWidth()
BridgeAR Jan 9, 2020
b1879aa
fixup
BridgeAR Jan 9, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
src: improve GetColumnWidth performance 
This improves the performance in GetColumnWidth for full width
characters.
  • Loading branch information
@BridgeAR
BridgeAR committed Jan 8, 2020
commit 9f7730ef99680ad4cd9452b704324db496016a8d
19 changes: 9 additions & 10 deletions src/node_i18n.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -728,16 +728,6 @@ static void ToASCII(const FunctionCallbackInfo<Value>& args) {
// Refs: https://github.com/KDE/konsole/blob/8c6a5d13c0/src/konsole_wcwidth.cpp#L101-L223
static int GetColumnWidth(UChar32 codepoint,
bool ambiguous_as_full_width = false) {
const auto zero_width_mask = U_GC_CC_MASK | // C0/C1 control code
U_GC_CF_MASK | // Format control character
U_GC_ME_MASK | // Enclosing mark
U_GC_MN_MASK; // Nonspacing mark
if (codepoint != 0x00AD && // SOFT HYPHEN is Cf but not zero-width
((U_MASK(u_charType(codepoint)) & zero_width_mask) ||
u_hasBinaryProperty(codepoint, UCHAR_EMOJI_MODIFIER))) {
return 0;
}

// UCHAR_EAST_ASIAN_WIDTH is the Unicode property that identifies a
// codepoint as being full width, wide, ambiguous, neutral, narrow,
// or halfwidth.
Expand All @@ -761,6 +751,15 @@ static int GetColumnWidth(UChar32 codepoint,
case U_EA_HALFWIDTH:
case U_EA_NARROW:
default:
const auto zero_width_mask = U_GC_CC_MASK | // C0/C1 control code
U_GC_CF_MASK | // Format control character
U_GC_ME_MASK | // Enclosing mark
U_GC_MN_MASK; // Nonspacing mark
if (codepoint != 0x00AD && // SOFT HYPHEN is Cf but not zero-width
((U_MASK(u_charType(codepoint)) & zero_width_mask) ||
u_hasBinaryProperty(codepoint, UCHAR_EMOJI_MODIFIER))) {
return 0;
}
return 1;
}
}
Expand Down