[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: enhanced debian-changelog-mode.el



Karl M. Hegbloom wrote:

> >>>>> "Peter" == Peter S Galbraith <GalbraithP@dfo-mpo.gc.ca> writes:
> 
>     Peter> A few _quick_ comments.  A diff to my installed version
>     Peter> shows you added menus and font-lock, so perhaps this line
>     Peter> is premature:
> 
>     Peter>  ;; This version has been heavily modified and enhanced by
>     Peter>  ;; Chris Waters
> 
>  Hmmm.  I think Chris intends to do more work on it, and just tapped
>  that down with inspired optimistic fervor in response to a mental
>  `dont forget to mention this isn't the original version'.

I didn't mean to sound too harsh, if that's what transpired from
my post.
 
>     Peter> Also, since you are _adding_ to existing code, it would be
>     Peter> polite to respect the indentation used: don't use hanging
>     Peter> closing parens, [...]
> 
>  I sometimes leave a hanging paren as a reminder to myself that there
>  may be more lines needed inside an expression later on.  Perhaps he
>  was just leaving it open for now like that, as an indicator to other
>  lisp coders reading it that "this is a place where more things might
>  go later on as this thing evolves".

Didn't think of that.  I hack elisp and know that some coders
prefer hanging parens as their style (and not to signify missing
content).

Peter


Reply to: