-
Notifications
You must be signed in to change notification settings - Fork 629
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.
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
Tex: parse some more latex macros #3239
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3239 +/- ##
==========================================
- Coverage 85.01% 84.98% -0.04%
==========================================
Files 206 207 +1
Lines 49127 49324 +197
==========================================
+ Hits 41765 41916 +151
- Misses 7362 7408 +46
Continue to review full report at Codecov.
|
I took much time to design Could you split dc57692 smaller ones as I wrote in #3236 (comment) ?
|
When the last parameter is optional and not present, the token is already read by parseWithStrategy() but this token is not reported to the caller as unprocessed which may cause missed tag for the following macro.
Macros like \newcommand can be written both as \newcommand{\foo}{bar} and \newcommand\foo{bar} Unify \def parsing and \newcommand parsing so that when strategy type is defined as '\\', the {} pair may or may not be present.
@masatake Done. |
Thank you very much. |
In addition to the macros already parsed, the old Geany readline-based parser also parses the following latex macros which may be useful:
\renewcommand
,\providecommand
\def\cmd{param}
\DeclareMathOperator
\newenvironment
,\renewenvironment
\newtheorem
This pul request adds these and introduces the corresponding kinds for them.