foster/denoland-deno
1
0
Fork 0
mirror of https://github.com/denoland/deno.git synced 2024-11-21 15:04:11 -05:00
Code Issues Packages 1 Wiki Activity

perf: optimize ByteString checks, hoist server rid getter (#19452)

Browse source 
Further improves preact SSR and express benches by about 2k RPS.

Ref https://github.com/denoland/deno/issues/19451
This commit is contained in:
Bartek Iwańczuk 2023-06-10 00:45:56 +02:00 committed by GitHub
parent f99a849a5f
commit 848cda619e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 12 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
ext/http/00_serve.js
View file

@ -682,8 +682,8 @@ function serveHttpOn(context, callback) {
// Run the server
const finished = (async () => {
const rid = context.serverRid;
while (true) {
const rid = context.serverRid;
let req;
try {
// Attempt to pull as many requests out of the queue as possible before awaiting. This API is

15
ext/webidl/00_webidl.js
View file

@ -59,7 +59,6 @@ const {
ReflectHas,
ReflectOwnKeys,
RegExpPrototypeTest,
RegExpPrototypeExec,
SafeRegExp,
SafeSet,
SetPrototypeEntries,
@ -403,11 +402,19 @@ converters.DOMString = function (V, prefix, context, opts = {}) {
return String(V);
};
// deno-lint-ignore no-control-regex
const IS_BYTE_STRING = new SafeRegExp(/^[\x00-\xFF]*$/);
function isByteString(input) {
for (let i = 0; i < input.length; i++) {
if (StringPrototypeCharCodeAt(input, i) > 255) {
// If a character code is greater than 255, it means the string is not a byte string.
return false;
}
}
return true;
}
converters.ByteString = (V, prefix, context, opts) => {
const x = converters.DOMString(V, prefix, context, opts);
if (RegExpPrototypeExec(IS_BYTE_STRING, x) === null) {
if (!isByteString(x)) {
throw makeException(
TypeError,
"is not a valid ByteString",