Skip to content

Commit 4dfd91a

Browse files
committed
Update opt-trace-format.md
1 parent a0e3522 commit 4dfd91a

1 file changed

Lines changed: 5 additions & 2 deletions

File tree

v3/docs/opt-trace-format.md

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,9 @@ more time elapsed since that date, the more likely things are
1515
out-of-date.
1616

1717
I'm assuming that you're competent in Python, JavaScript, command-line-fu, and Google-fu,
18-
and general [command-line BS](http://www.pgbovine.net/command-line-bullshittery.htm),
19-
so I won't do much hand-holding in these directions.
18+
and [command-line BS](http://www.pgbovine.net/command-line-bullshittery.htm).
19+
You may still be confused about details after reading this doc, so
20+
feel free to email philip@pgbovine.net if you have questions.
2021

2122

2223
## Trace Overview
@@ -812,3 +813,5 @@ for i in enumerate(choices):
812813
```
813814

814815
Thus, this is one target for optimization if deemed necessary down the line. However, I'm hesitant to optimize the trace since it makes trace-handling code in the frontend and backend more complicated :/
816+
817+
(One elegant way to optimize is to simply store the completed `stdout` buffer ONCE in the trace. And then within each execution point, store a byte index into the buffer. Thus, to render the stdout contents for each point, all we do is slice the buffer from the start to that index.)

0 commit comments

Comments
 (0)