Comments (4)
Hi @raystubbs,
What behavior would you expect in this scenario? An index of zero is indeed out of range for a zero-length array.
Thanks!
from clearscript.
I'd expect it not to throw when copying 0 bytes with offset
or sourceIndex
as 0
.
from clearscript.
I'd expect it not to throw when copying 0 bytes with
offset
orsourceIndex
as0
.
So, if count
is zero, the method should just return zero without checking the other arguments? While that would certainly be a simple change, we're not sure it would be more correct from an API perspective (unless there's precedent).
In any case, if that's the behavior you want, would it not be possible to check the count before invoking WriteBytes
?
from clearscript.
Alright, I can't think of a particular example/precedent off the top of my head. But the fact that this behavior was surprising should probably be a hint in itself.
And we did change our code to do an explicit check. Just seems like it shouldn't be necessary.
Anyway thanks for the quick responses. If it isn't obvious to you that this isn't good, then probably no reason to change it. But may be worth noting the behavior in the docs.
from clearscript.
Related Issues (20)
- Question: try pattern from script HOT 2
- Behavior of the `MaxRuntimeHeapSize` property of `V8ScriptEngine` class has changed HOT 2
- Debugging ClearScript with Visual Studio Code HOT 1
- Control of the ClearScript Engine HOT 4
- "RangeError: Array buffer allocation failed" when Executing "new Uint8Array()" HOT 2
- Allow configuration to limit the number of max statements to be executed HOT 13
- How to change the local timezone used by the engine's date functions HOT 4
- VBScript will be discontinued, can clear script be a solution? HOT 2
- Issue with calling JavaScript functions defined as function from ClearScript in C# HOT 3
- [FR]: Expose V8's CompileFunction (Along with the Rest of v8::ScriptCompiler) HOT 5
- Feature Request: Add the Deno engine. HOT 5
- ReadOnly ScriptMember still writable according to getOwnPropertyDescriptor HOT 2
- Runtime support appears incorrect HOT 2
- License ambiguity between git repo and NuGet HOT 2
- Allow for AwaitDebugger but don't pause on first HOT 3
- VBScriptEngine: Changing object with AddHostObject doesn't work HOT 2
- I encountered some issues while interacting with Turf and may need assistance HOT 5
- Possible bug in "FOR EACH item IN list" loop ? HOT 4
- Engine in VB.NET returning "nothing" to host HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from clearscript.