connectmodule/endconnectmodule support

[danmcmahill]

AMS defines connectmodule/endconnectmodule which work pretty much the same way as module/endmodule but for connect modules instead of normal modules. This adds support to get proper syntax highlighting, indentation, and adding of comments.

I was only able to test with emacs (GNU Emacs 26.1) as an unmodified clone failed to build under xemacs (21.5).

[wsnyder]

Thanks for providing a patch for this.

• It looks like the only tests you modified were comments, please provide some tests that use connectmodule/endconnectmodule to check the indentation and anything else you feel is important.
• (defconst verilog-keywords: Please add connectmodule/endconnectmodule in a separately commented section mentioning AMS.
• (verilog-ams-keywords: Please add endconnectmodule
• (verilog-font-general-keywords: No changes here as is in verilog-ams-keywords

If you search for module (sometimes M is not part of it) it looks like there's spots that should have been patched also:

• (defvar verilog-imenu-generic-expression
• (defconst verilog-defun-level-not-generate-re
• (setq verilog-font-lock-keywords-1
• (defun verilog-backward-sexp
• (defun verilog-forward-sexp
• (defun verilog-in-case-region-p
• (defun verilog-in-generate-region-p
• (defun verilog-calculate-indent
• (defun verilog-indent-declaration
• (defun verilog-func-completion
• (defun verilog-showscopes
• (defun verilog-module-inside-filename-p
• (defun verilog-inject-arg

[danmcmahill]

Thanks Wilson. I think I've addressed the issues you pointed out. I added two simple checks. One checks that indenting works and the other makes sure labeling of endconnectmodule // <name> works. Those are likely the biggest care abouts. The good news is connect modules tend to be relatively short and without hierarchy. But having indenting work is huge.

[wsnyder]

Merged, thanks for this.