-
Notifications
You must be signed in to change notification settings - Fork 37
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
Remove Base64Parsing #259
Remove Base64Parsing #259
Conversation
pjfanning
commented
Jul 23, 2023
•
edited
Loading
edited
- use parboiled class instead (we have removed all the other parboiled code, now that we have a jar dependency for it)
- the code that I moved to Http2SettingsHeader did not come from the parboiled Base64Parsing class
- relates to add parboiled2 license #257
@@ -42,4 +48,5 @@ private[pekko] object Http2SettingsHeader { | |||
val reader = new io.ByteStringParser.ByteReader(bytes) | |||
Http2FrameParsing.readSettings(reader, log) | |||
} | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove this new line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does the removed method been used anywhere ,eg in grpc module?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the code that is being removed is package private code so it would be really bad practice to use it outside of this module
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've just checked incubator-pekko-grpc, incubator-pekko-connectors and incububator-pekko-management git repos and can't find any uses of the class that I am removing in this PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've committed a change that removes the new line
Copyright © 2009-2013 Mathias Doenitz <http://parboiled2.org> | ||
Copyright © 2013 Alexander Myltsev | ||
|
||
--------------- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
only added yesterday - but not needed if we remove Base64Parsing.scala
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM