Hi, sorry for my late response.

At Thu, 20 Jun 2013 09:51:18 +1000,
Kevin Ryde <use...@zip.com.au> wrote:
> 
> /usr/share/emacs/site-lisp/howm/howm-misc.el contains an auto-mode-alist
> entry
> 
>     ("\\.howm$" . text-mode)
> 
> This could helpfully be in 50howm.el so that .howm files are always
> visited in text-mode, irrespective whether you have loaded howm yet or
> not.  It can be outside the package-dir existence test too, since it
> will still work under a "dpkg --remove" when only 50howm.el is left.
> 
> The regexp usually should be
> 
>      "\\.howm\\'"
> 
> so as to match only end-of-string, not a newline.  Not that filenames
> ought to contain newlines, but just in case.

Thanks. I'll fix it.

> Oh, and I'd suspect there's no need to append to auto-mode-alist the way
> howm-misc.el has.  It might as well go at the start since there won't be
> anything else setting up .howm files.  A plain add-to-list would avoid
> duplicating an entry when in both 50howm.el and howm-misc.el.

We don't provide auto-mode-alist in 50howm.el, and upstream provide
auto-mode-alist in howm-misc.el. Thus I follow upstream manner.

---
Youhei SASAKI <uwab...@gfd-dennou.org>
              <uwab...@debian.or.jp>
GPG fingerprint:
  4096/RSA: 66A4 EA70 4FE2 4055 8D6A C2E6 9394 F354 891D 7E07


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to