-
Notifications
You must be signed in to change notification settings - Fork 0
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
Human-readable description of the decoding steps #2
Comments
Here's my go at what the description for the steps from the parent comment might look like:
The values in parentheses are "blanks". The sentence needs to work with an arbitrary "identifier" put there. There's three categories of identifiers that we can encounter, and here's how I would put those into text:
I'm pretty happy with this concept, I think that should work for just about anything and it reads okay. |
Oh, actually, I think we might need to abandon the idea of replacing temporary variable names with "the return value of line n". I'm not sure whether that's possible since we allow writing to arbitrarily nested paths within variables. Consider (a bit contrived, but just to demonstrate what I'm thinking): b = base64Decode(body)
b.a.c = something(b.a.c)
b = parseJson(b)
b.a.c = parseQueryString(b.a.c)
res.body.b.a = something(b.a) Generating a description for But for While I'm quite sad about that, I think we need to stick with "the variable |
OK, the very first (and basic) implementation is done in 9e0c572: Currently, the identifiers are just pasted in as-is. |
Mostly fixed with #1 but translations are still pending. |
We need to explain the decoding steps on the site. In #1, I have already implemented a pseudo code rendering of them:
But we also want to describe them as continuous text.
The text was updated successfully, but these errors were encountered: