Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Parsing the arguments of \def as separate objects #131

Open
SengerM opened this issue Jul 30, 2021 · 0 comments
Open

Parsing the arguments of \def as separate objects #131

SengerM opened this issue Jul 30, 2021 · 0 comments

Comments

@SengerM
Copy link

SengerM commented Jul 30, 2021

The output of this

import TexSoup

latex = r'\def\lagrangian{\mathscr{L}}'

soup = TexSoup.TexSoup(latex)

print(soup.contents)

is

[\def{\}{lagrangian}, {\mathscr{L}}]

Is this the correct behavior? Sorry if not, I am not a LaTeX expert. But to me it is more logical to have everything parsed into the same object.
masonproffitt added a commit to masonproffitt/TexSoup that referenced this issue Dec 3, 2021
@masonproffitt
fix \def without braces around command alvinwan#118 alvinwan#131
2a19a56 
e.g., \def\commandname{command text}
@masonproffitt masonproffitt mentioned this issue Dec 3, 2021
Merged
masonproffitt added a commit to masonproffitt/TexSoup that referenced this issue Dec 6, 2021
@masonproffitt
improve \def parsing alvinwan#118 alvinwan#131
f72ef08 
Don't surround new command with braces, and make sure it's parsed as a TexCmd.
This makes \def more consistent with \newcommand.
alvinwan pushed a commit that referenced this issue Feb 4, 2023
@masonproffitt
Fix parsing of \def without braces around new command name (#132)
ca6d93c 
* fix \def without braces around command #118 #131

e.g., \def\commandname{command text}

* improve \def parsing #118 #131

Don't surround new command with braces, and make sure it's parsed as a TexCmd.
This makes \def more consistent with \newcommand.

* fix required arguments parsing logic
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@SengerM