-
Notifications
You must be signed in to change notification settings - Fork 81
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
What's next... #5
Comments
You mean released 1.8.1.2 or 1.8.2 that one which is in dev branch? Just to be sure :) Also, I suppose you want 1.0.0-alpha be compatible with 1.8.1(.2?) that you initially picked up, but later anyway plan to upgrade to later version of lz4 with those breaking changes (imho there is nothing wrong in breaking changes, user always can stay with old/previously released version)? |
1.8.2 fixes some low probability/high impact security issue. It won't happen on PC but might happen on IoT devices. So, I'm not too concerned but still it is worth to port it, as once ported it freezes for a while. |
@MiloszKrajewski got it. :) if there is no problem for PC, so no reason to rush then with bringing 1.8.2 update :) |
@MiloszKrajewski development stalled? :-\ |
Yup. I had a bug and a mental block for a while. And then, after returning from holiday, found and fixed problem in one day. |
@MiloszKrajewski good to see you back. |
I just release to nuget and wrote some docs on main page. I'll wait for some API feedback and try to port next version. As long as it is just bug fixes it should be relatively easy, but if there are breaking changes it may take while. |
@MiloszKrajewski Well, since major version hasn't been changed, I guess its just minor fixes + optimizations, so I guess it shouldn't take much time :) |
Seems like development a bit stalled, but curious, any progress on this? |
@MiloszKrajewski ping :) |
@MiloszKrajewski Hello Milosz, when you plan to continue development on this library? Or its no longer maintained? |
Yes, I do. I'm just doing some online courses now and all my evenings are consumed by writing Scala. I'll come back to this project soon. |
Glad to hear :)
Nah, not at the moment at least :) |
@MiloszKrajewski |
I pulled 1.9.1 and compared it with 1.8 and change are quite drastic so no tweaking, it would require full port again. On the radar, but functional changes (like true async/await) would take precedence I think. |
Its better than nothing at all :)
Oh, good. |
@MiloszKrajewski |
New version 1.2.1-beta released |
I see 'add true async read/write interface' in vNext above, but I also wrote up #34 to explain the value to me. Without a fix, I'll have to look for other fast compression options. Thanks. |
1.0.3
Position
andLength
(if available)1.1.0
1.1.3
1.2.1-beta
1.2.5
Position
andLength
for compression stream (nothing fancy, just symmetry)async
read/write interfaceasync
support (ie: DisposeAync for .NET Standard 2.1)1.3.0
1.3.3
vNext
Not planned
goto
s across scopes, not a thing in C#)Note: block checksum and content checksum are not required (and may be ignored)
Note: frame encoder/decoder now pulls data from stream, which means it needs to "understand" async. I would love to remove this dependency, and make it a pure state machine, which will still allow to build "async" solution on top of it.
The text was updated successfully, but these errors were encountered: