search

veripool / verilog-mode

Verilog-Mode for Emacs with Indentation, Hightlighting and AUTOs. Master repository for pushing to GNU, verilog.com and veripool.org.
http://veripool.org/verilog-mode
GNU General Public License v3.0
253 stars 90 forks source link

Current Mode of Disabling of AUTOs when expansion fails #1676

Closed engrvns closed 4 years ago

engrvns commented 4 years ago

A suggestion on auto disabling of erroring AUTOs.

In the verilog-mode version toda, when a specific AUTO expansion fails for any reason, the AUTO gets disabled by replacing the AUTO as -

/ module AUTO_TEMPLATE ( ... < - Assume some user error /

/ module /-AUTO_TEMPLATE-/ ( <- AUTO_TEMPLATE disabled ... /

The characters used to disable is very subtle and easy to miss if you are not aware it.. It's very close to a user intentionally disabling an AUTO by intention.

Instead can the disable be something that can pops up and easy to search say for example -

/module /AUTO_TEMPLATE_ERROR/ ( or / module /AUTO_TEMPLATE_FIXME/ (

This strings ERROR/FIXME draws attention to the user. Or can this be a verilog-mode variable?

Thanks, engrgvns

wsnyder commented 4 years ago

The autos don't ever mark your code with errors as you indicate. What does /-* is verilog-comment-region, which is only invoked by the user when they want to comment something out manually, never by autos.