Skip to content

COMP: Fix multi-line comment warning generated by latex notation#446

Merged
blowekamp merged 1 commit into
InsightSoftwareConsortium:masterfrom
phcerdan:guide_updateTransformIndexToPhysicalPoint
Jan 25, 2019
Merged

COMP: Fix multi-line comment warning generated by latex notation#446
blowekamp merged 1 commit into
InsightSoftwareConsortium:masterfrom
phcerdan:guide_updateTransformIndexToPhysicalPoint

Conversation

@phcerdan

Copy link
Copy Markdown
Contributor

Create a global definition to use inside the matrix environments,
instead of \\.

\gdef\NL{\\}

Reference:
https://tex.stackexchange.com/questions/234309/redefining-line-break-double-backslash-for-align-environment

Create a global definition to use inside the matrix environments,
instead of `\\`.
```latex
\gdef\NL{\\}
```

Reference:
https://tex.stackexchange.com/questions/234309/redefining-line-break-double-backslash-for-align-environment
@phcerdan
phcerdan requested a review from thewtex January 25, 2019 18:00
@phcerdan

Copy link
Copy Markdown
Contributor Author

image

Looking good!

@blowekamp

blowekamp commented Jan 25, 2019

Copy link
Copy Markdown
Member

Hooray! I see a warning in my PR and there is already a patch!

@thewtex thewtex left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @phcerdan !

@blowekamp
blowekamp merged commit e575a91 into InsightSoftwareConsortium:master Jan 25, 2019
hjmjohnson pushed a commit to hjmjohnson/ITK that referenced this pull request May 6, 2026
…updateTransformIndexToPhysicalPoint

COMP: Fix multi-line comment warning generated by latex notation
hjmjohnson pushed a commit to hjmjohnson/ITK that referenced this pull request May 12, 2026
…updateTransformIndexToPhysicalPoint

COMP: Fix multi-line comment warning generated by latex notation
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.

3 participants