Skip to content

Commit

Permalink
Clarify why we ENABLE_COPYWITHIN is false
Browse files Browse the repository at this point in the history
  • Loading branch information
samhed committed Sep 25, 2019
1 parent 3aeaea5 commit c90d535
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions core/websock.js
Expand Up @@ -17,6 +17,8 @@ import * as Log from './util/logging.js';
// this has performance issues in some versions Chromium, and
// doesn't gain a tremendous amount of performance increase in Firefox
// at the moment. It may be valuable to turn it on in the future.
// Also copyWithin() for TypedArrays is not supported in IE 11 or
// Safari 13 (at the moment we want to support Safari 11).
const ENABLE_COPYWITHIN = false;
const MAX_RQ_GROW_SIZE = 40 * 1024 * 1024; // 40 MiB

Expand Down

0 comments on commit c90d535

Please sign in to comment.