[Matlab-emacs-discuss] [a problem in the branch: the return key] (was: add-change-log-entry-other-window for matlab files)

[Uwe B. <ou...@ma...>]

>>> "EL" == Eric Ludlam <eri...@gm...> writes:


> Sounds like a fine idea to me.  I posted a change to the hairyblocks
> branch that will use the function name, or text after the cell start
> as the 'defun' for add log.

> The hairyblocks branch has been stable for a while with a lot of
> developers at MW using it.  It should be ready to merge into the main
> branch whenever you are ready for it.



I am just running into a problem with the hairyblocks branch.

I want to insert an empty line, in a matlab.m file hitting 
the return key, but I receive the following error 

,----
| Debugger entered--Lisp error: (void-function matlab-return)
|   signal(void-function (matlab-return))
|   help-fns--analyze-function(matlab-return)
|   help-fns-function-description-header(matlab-return)
|   describe-function-1(matlab-return)
|   describe-key((([return] . [return])))
|   funcall-interactively(describe-key (([return] . [return])))
|   call-interactively(describe-key nil nil)
|   command-execute(describe-key)
`----

Indeed my personal binding had (I don't know for how long)
  (local-set-key [return]  'matlab-return) 


Eyeballing the changelog, I find 
  Remove matlab-comment-indent as a setting (use nil)
│  (matlab-return-function, matlab-return, matlab-plain-ret)
│  (matlab-indent-after-ret, matlab-indent-end-before-ret)
│  (matlab-semicolon-on-return, matlab-indent-before-ret)
│  (matlab-linefeed):

Which I interpret that you removed matlab-return.

So I suggest that before I merge that branch into master, we had into
the NEWS file and entry were we tell which functions have been removed
in order to avoid users complains.

What do you think?

Uwe