Add source position tracking for grid tables#883
Conversation
|
|
||
| ContinueProcessingLine = true; | ||
|
|
||
| ResetLine(newLine, 0); |
There was a problem hiding this comment.
| ResetLine(newLine, 0); | |
| ResetLine(newLine, column); |
Shouldn't it pass column?
|
Sorry about that — the first fix for this issue was a bit of a hack, and part of that code was left behind. It shouldn't have affected functionality, but it wasn't clean. I've corrected it now, and it should be fine. Thanks for spotting it! |
|
Since I'm not the original author, my contribution might not be entirely ideal. |
|
Thanks for this. As I have been quite away from the details on this project, I will have to double check things, but it looks ok. 🙂 |
Fixes #881
Adds source position tracking for grid tables and corresponding tests.