From ce61c1765a74b2dde680ab01c45c0f16564c8ca4 Mon Sep 17 00:00:00 2001 From: CJ van den Berg Date: Mon, 13 Apr 2026 22:09:08 +0200 Subject: [PATCH] refactor: use changes_when_lowercased consistently in toggle_case switch_case already uses changes_when_lowercased to detect uppercase characters. toggle_case was using is_lowercase with inverted branch order, which is functionally equivalent but inconsistent. --- src/buffer/unicode.zig | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/buffer/unicode.zig b/src/buffer/unicode.zig index fd13e91d..fc78be54 100644 --- a/src/buffer/unicode.zig +++ b/src/buffer/unicode.zig @@ -208,10 +208,10 @@ pub fn toggle_case(allocator: std.mem.Allocator, text: []const u8) TransformErro const view: Utf8View = .initUnchecked(text); var it = view.iterator(); while (it.nextCodepoint()) |cp| { - const cp_ = if (uucode.get(.is_lowercase, cp)) - uucode.get(.simple_uppercase_mapping, cp) orelse cp + const cp_ = if (uucode.get(.changes_when_lowercased, cp)) + uucode.get(.simple_lowercase_mapping, cp) orelse cp else - uucode.get(.simple_lowercase_mapping, cp) orelse cp; + uucode.get(.simple_uppercase_mapping, cp) orelse cp; var utf8_buf: [6]u8 = undefined; const size = try utf8Encode(cp_, &utf8_buf); try writer.writeAll(utf8_buf[0..size]);