Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Signal pdu #4

Closed
wants to merge 10 commits into from
Closed

Signal pdu #4

wants to merge 10 commits into from

Conversation

seanidzenga
Copy link

Completed implementation of the signal pdu - rather than using many nByteChunk functions I've implemented a Chunk class which takes a parameter for however many bytes you need.
new oneByteChunk() == new Chunk(1)
new sixByteChunk() == new Chunk(6)
etc...

Sean Idzenga added 10 commits September 2, 2016 15:43
…ith 64-bit integers (no arithmetic, unfortunately)

Completed InputStream.prototype.readLong()
…on to and from a buffer. readLong() and writeLong() implemented now.
…to determine how many oneByteChunks are to be added to this.data
…lass takes a single argument chunkSize to determine how many bytes long it should be, ie 2 produces a "TwoByteChunk" - usage doesn't change much:

var foobar = new Chunk(4);
foobar.initFromBinaryDIS(inputStream);
this.data.push(foobar.otherParameters);

Refactoring already complete.
…ding, the byte arrays are not parameters. Changed to simply read "data".
Missed three instances of OneByteChunk() corrected to use dis.Chunk(1)
@leif81
Copy link

leif81 commented Nov 29, 2019

@seanidzenga i merged this into the new home for this repository.

https://github.com/open-dis/open-dis-javascript

Feel free to close this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants