Readability suggestion #377
Comments
|
(or maybe even just drop The code is and let the user add that if they want) |
|
The getCode interface seems to be doing what you have in mind. Is it missing something? |
|
Yes, at the minimum, a line break so that the code starts on column one.
… On Feb 16, 2022, at 1:41 PM, Vassil Vassilev ***@***.***> wrote:
The getCode interface seems to be doing what you have in mind. Is it missing something?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.
|
|
@Nirhar, can you look into this issue, too? |
Nirhar
added a commit
to Nirhar/clad
that referenced
this issue
May 25, 2022
Previously the dump function printed the code immediately after "The code is" which made the actual code less readable. Now a '\n' character has been added after "The code is" so that the actual code starts from a new line, making it more readable. This patch fixes vgvassilev#377
vgvassilev
pushed a commit
that referenced
this issue
May 25, 2022
Previously the dump function printed the code immediately after "The code is" which made the actual code less readable. Now a '\n' character has been added after "The code is" so that the actual code starts from a new line, making it more readable. This patch fixes #377
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
clad/include/clad/Differentiator/Differentiator.h
Line 201 in 4f1c10e
would be more readable with another line break
The text was updated successfully, but these errors were encountered: