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
Bparser integration + Stream construction/queueing #47
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
TODO: timeouts
dr7ana
force-pushed
the
bparser-testing
branch
2 times, most recently
from
September 20, 2023 18:16
8b96513
to
49c28e8
Compare
- bparser sandboxing updated to use stream constructor methods from PR#35 - differentiated request/response/command/error transmission - updated 002 test case to proof of concept for command and request/response
dr7ana
force-pushed
the
bparser-testing
branch
from
September 20, 2023 18:30
49c28e8
to
ecb56c6
Compare
tewinget
reviewed
Sep 21, 2023
jagerman
reviewed
Sep 21, 2023
jagerman
reviewed
Sep 21, 2023
jagerman
reviewed
Sep 21, 2023
dr7ana
force-pushed
the
bparser-testing
branch
from
September 21, 2023 21:37
c382698
to
b38cca4
Compare
jagerman
reviewed
Sep 21, 2023
dr7ana
force-pushed
the
bparser-testing
branch
from
September 21, 2023 21:51
b38cca4
to
7155176
Compare
jagerman
reviewed
Sep 21, 2023
jagerman
reviewed
Sep 21, 2023
jagerman
reviewed
Sep 21, 2023
dr7ana
force-pushed
the
bparser-testing
branch
from
September 21, 2023 22:10
7155176
to
8e38f5f
Compare
dr7ana
changed the title
Bparser integration
Bparser integration + Stream construction/queueing
Sep 21, 2023
jagerman
reviewed
Sep 21, 2023
dr7ana
force-pushed
the
bparser-testing
branch
2 times, most recently
from
September 22, 2023 15:44
b43aa48
to
ac72e35
Compare
jagerman
reviewed
Sep 22, 2023
jagerman
reviewed
Sep 22, 2023
jagerman
reviewed
Sep 22, 2023
jagerman
reviewed
Sep 22, 2023
jagerman
reviewed
Sep 22, 2023
jagerman
reviewed
Sep 22, 2023
jagerman
reviewed
Sep 22, 2023
jagerman
reviewed
Sep 22, 2023
jagerman
reviewed
Sep 22, 2023
jagerman
reviewed
Sep 22, 2023
jagerman
approved these changes
Sep 22, 2023
dr7ana
force-pushed
the
bparser-testing
branch
from
September 22, 2023 17:05
ac72e35
to
86a7a7b
Compare
dr7ana
force-pushed
the
bparser-testing
branch
from
September 22, 2023 17:10
86a7a7b
to
3af542c
Compare
jagerman
reviewed
Sep 22, 2023
- deque of sent_requests plus modified binary search to find RID's - bparser callback only exposes message type - ::respond method implemented in message type exposed in callback - message type holds weak_ptr to bparser
dr7ana
force-pushed
the
bparser-testing
branch
from
September 22, 2023 17:29
3af542c
to
aa678fb
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Bparser is now implemented via the developed custom stream creation methods. New stream creation methods are implemented to queue and construct user-defined extensions of streams