Repository navigation
Conversation
Add ANSYS APDL and NASTRAN highlighting, folding, and column guides
RaiKoHoff
left a comment
There was a problem hiding this comment.
Non standard Lexilla lexers
|
Your pull request appears to be very well engineered, and we sincerely appreciate the time and effort you have invested in it. Unfortunately, one of our current goals is to reduce the number of custom ("home-grown") lexers (lexer_x) and rely as much as possible on the lexers provided by Lexilla. This helps us keep the codebase maintainable and reduces our long-term maintenance burden. For this reason, we regret that we cannot accept your pull request. However, if you are interested in maintaining your own version of Notepad3, including support for ANSYS APDL and NASTRAN, in your GitHub repository, we would be very happy to reference your project on our website. This would allow users interested in these additional lexers to discover and benefit from your work. Thank you again for your valuable contribution, your understanding, and the effort you have put into this. |
Hi RaiKoHoff, Thank you for reviewing my contribution and for explaining the project’s direction. I understand the goal of reducing custom lexers and the associated maintenance burden. After checking the upstream sources, I noticed that Lexilla already provides an APDL lexer in lexers/LexAPDL.cxx (SCLEX_APDL). Notepad3’s current lexer catalogue does not register it. My submitted PR added custom implementations under lexers_x, so your concern about the current implementation is valid. Would you be open to a smaller, separate PR that enables APDL using the existing upstream Lexilla lexer, with only the necessary build/catalogue integration, syntax styles, keyword lists, and language selection/file associations? I would leave the custom APDL implementation, NASTRAN support, and column-guide features out of that proposal. Any APDL lexer improvements needed beyond the existing upstream implementation could be proposed to Lexilla first. For NASTRAN, I found Lexilla’s MSC Nastran DMAP lexer, but that targets a different language from the BDF/NAS input files supported by my PR. I therefore understand that it does not resolve your concern about the custom NASTRAN lexer. My aim is to make these engineering text formats easier to edit while following Notepad3’s preference for upstream-maintained lexers. Would an APDL-only contribution along these lines fit the project’s goals? Thank you again for your time and for your kind offer to reference my fork. |
Summary
Add editing support for ANSYS APDL and NASTRAN input files, including syntax highlighting and code folding.
Validation
8c96c2185481bcecded3a41e2c90b3de998fe604; source head is9290c8023a2279a707e69b7185e462dc946b07c2.git diff --checkreports trailing spaces on four NASTRAN fixture lines. The fixture instructions require preserving fixed-column bytes.Screenshots