Remove a use of String16::remove.

This function, ironically, is being removed. This is one of only two
users, and it's pretty hard to use correctly. In this case, using one
of the existing constructors to keep a prefix seems clearer even in a
world where remove() was better implemented.

Test: treehugger
Change-Id: Icdf02d9fcc059c396c13787d9bb4e8bda5658656
This commit is contained in:
Elliott Hughes
2021-05-11 16:24:35 -07:00
parent 61f27d7d52
commit 0523328a00

View File

@@ -194,7 +194,8 @@ static String16 pseudo_generate_expansion(const unsigned int length) {
break;
}
}
result.remove(length + ext, 0);
// Just keep the first length + ext characters
result = String16(result, length + ext);
}
return result;
}