Skip to content

Conversation

pzirnik
Copy link

@pzirnik pzirnik commented Jun 5, 2025

cleaned up the patch from issue#5464, please review.

@dilyanpalauzov
Copy link
Contributor

I am not sure how will this work with the other protocols supported by imtest (which installs as symlinks), but I think it is okay. I also guess that modifying str is okay, which can be verified by running sivtest under valgrind with many responses by the server (I have not verified it).

Why doesn't str jump to the byte after the first "?

You could add a check on the last else whether the first byte is { and emit an error, if it is not.

N.B. Progress here does not depend on me and opinions, opposite to what I said, are possible.

@dilyanpalauzov
Copy link
Contributor

Will this work? :

Closes: #5464.

@dilyanpalauzov
Copy link
Contributor

Will this work? :

Closes: #5464.

It does not work, at least not, when it is from me.

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.

2 participants