Look at collapsing MutableByteSequence into ArrayByteSequence

Question

Look at collapsing MutableByteSequence into ArrayByteSequence

dlmarion opened this issue 2 months ago · 4 comments

MutableByteSequence extends ArrayByteSequence with two method to modify the object, but ArrayByteSequence isn't immutable. MutableByteSequence is only used in 2 classes. Evaluate whether or not MutableByteSequence can be removed in favor of ArrayByteSequence with necessary modifications.

Answer 1 · 2024-07-10T19:01:36.000Z

MutableByteSequence is not in a public API package so it would be easy to delete

Answer 2 · 2024-07-15T15:46:24.000Z

I can look into this.

Answer 3 · 2024-07-22T11:54:46.000Z

@meatballspaghetti - Is there a pull request for your commit to merge it into the main repo?

Answer 4 · 2024-07-22T12:19:05.000Z

@meatballspaghetti - Is there a pull request for your commit to merge it into the main repo?

Yes, there is now #4745 .