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.
This commit is contained in:
CJ van den Berg 2026-04-13 22:09:08 +02:00
parent b2cb003d82
commit ce61c1765a
Signed by: neurocyte
GPG key ID: 8EB1E1BB660E3FB9

View file

@ -208,10 +208,10 @@ pub fn toggle_case(allocator: std.mem.Allocator, text: []const u8) TransformErro
const view: Utf8View = .initUnchecked(text); const view: Utf8View = .initUnchecked(text);
var it = view.iterator(); var it = view.iterator();
while (it.nextCodepoint()) |cp| { while (it.nextCodepoint()) |cp| {
const cp_ = if (uucode.get(.is_lowercase, cp)) const cp_ = if (uucode.get(.changes_when_lowercased, cp))
uucode.get(.simple_uppercase_mapping, cp) orelse cp uucode.get(.simple_lowercase_mapping, cp) orelse cp
else else
uucode.get(.simple_lowercase_mapping, cp) orelse cp; uucode.get(.simple_uppercase_mapping, cp) orelse cp;
var utf8_buf: [6]u8 = undefined; var utf8_buf: [6]u8 = undefined;
const size = try utf8Encode(cp_, &utf8_buf); const size = try utf8Encode(cp_, &utf8_buf);
try writer.writeAll(utf8_buf[0..size]); try writer.writeAll(utf8_buf[0..size]);