forked from veripool/verilog-mode
-
Notifications
You must be signed in to change notification settings - Fork 0
/
TODO
65 lines (48 loc) · 2.25 KB
/
TODO
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
# DESCRIPTION: To-do list for verilog-mode
######################################################################
Occasionally:
* M-x checkdoc
* Make sure it's as byte compile clean as possible.
Features
Make a hiearchy map of the current module and below in a spare buffer.
;From: Dan Nicolaescu <[email protected]>
;Date: Sun, 14 Oct 2007 10:49:43 -0700
; > (defvar verilog-mode-map ()
; > "Keymap used in Verilog mode.")
; > (if verilog-mode-map
; > ()
; > (setq verilog-mode-map (make-sparse-keymap))
; > (define-key verilog-mode-map ";" 'electric-verilog-semi)
; Better use:
; (defvar verilog-mode-map
; (let ((map (make-sparse-keymap)))
; (define-key map ";" 'electric-verilog-semi)
; ... etc etc
From: Stefan Monnier <[email protected]>
Date: Sat, 13 Oct 2007 22:41:15 -0400
- Rather than create a new syntax-table in each buffer, just do
(defvar verilog-mode-syntax-table
(let ((st (make-syntax-table)))
(modify-syntax-table ...)
...
st))
once and forall (inlining the only calls to verilog-setup-dual-comments
and verilog-populate-syntax-table).
Regarding verilog-setup-dual-comments, even if you consider the XEmacs
solution cleaner, the Emacs solution is not only equivalent but it also
works under XEmacs, so you can use it everywhere and remove an
emacs-version check.
- You can then remove the verilog-emacs-features abomination.
- The part
(require 'verilog-mode)
is pretty hideous and deserves either to be fixed (i.e. removed), or large
comments justifying each part.
Also some of the subsequent requires are unneeded (e.g. the `imenu' one).
- Use easy-menu-add rather than add-submenu so it works on both Emacs
and XEmacs.
- Emacs-21's comment-region and uncomment-region should work correctly
(including mangling nested comments, although in a different way from the
one you chose).
verilog-comment-regiont and verilog-uncomment-region
can (probably) be dropped when running under Emacs>=21 (and the C-c C-c
binding could also be removed if you assume your users will use M-;)