[WIP] Markdown reader: extension to insert source code position attributes #4659
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.
Optionally inserts source-code position attributes for
CodeBlock
in Markdownsources. The extension is
Ext_code_block_source_position
.This reads the parser position (
SourcePos name line col
) at the beginning ofparsing a
CodeBlock
and adds the attributes[("sourceName", name), ("sourceLine", line), ("sourceColumn", col)]
if theextension is turned on.
Addressed #4657
WIP Part
Currently, the
sourceName
is returning nothing useful (just the wordsource
). It would be nice if it actually returned the name of the source file for generating things like line directives for files generated from multiple literate sources: https://gcc.gnu.org/onlinedocs/cpp/Line-Control.htmlAny ideas @jgm? I could either (1) omit the
sourceName
attribute, or (2) figure out what the original file name is (usingstdin
or-
for standard input). I would prefer(1)(2) for the above reason.