Re: [Matlab-emacs-discuss] [matlab-cell] (was: Github Migration?)

[Nidish N. B. <nid...@il...>]

>>>> "NNB" == Nidish Narayanaa Balaji<nid...@il...>  writes:
> Hi
>
>
>> A few pointers:
>
>> The main features that the new minor-mode enables are:
>>    * It redefines the page-break character to "%%" so an overline is
>>      drawn over those, making it visually look better.
>>    * The current cell that point is in is highlighted by being boldfaced
>>      (this is customizable).
>>    * Convenience functions for moving point forward and backward by one
>>      cell (also support being called with prefix arguments).
>>    * Convenience functions for moving a cell up or down by a block
>>      (number-prefixes supported too).
> After a very very brief testing, some comments:
>
>      1. Could you please include matlab-cell in the Makefile?
>
>      2. So I understand your code does not generalise in any way
>         matlab-shell-run-cell?

 1.   Done. I've added it to the lisp_LISP variable in the Makefile and
    can confirm that it builds fine.

 2. This is correct. I found no reason to rewrite matlab-shell-run-cell
    since it works fine as is. Let me know if you feel otherwise.

>>    * Some more non-interactive convenience functions to help development.
>>    * I've not bound any of the functions to any keymap within the package
>>      yet.
>>      In my config though, these are the bindings I'm using (in case
>>      someone's interested):
>>                   ("C-v" . 'matlab-cell-forward-cell)
>>                   ("M-v" . 'matlab-cell-backward-cell)
>>                   ("C-<return>" . matlab-shell-run-cell)
>>                   ("C-c C-b" . (lambda () (interactive)
>>                                  (save-window-excursion
>>                                    (matlab-shell-run-cell))
>>                                  (matlab-cell-forward-cell)))
>>                   ("C-x n c" . 'matlab-cell-narrow-to-cell)
>>                   ("M-[" . 'backward-paragraph)
>>                   ("M-]" . 'forward-paragraph)
>>                   ("C-c <down>" . 'matlab-cell-move-cell-down)
>>                   ("C-c <up>" . 'matlab-cell-move-cell-up)
>> I'll appreciate comments and suggestions! :)
> Could you maybe enhance the documentation of matlab-cell-mode, for
> example describe the navigation functions.
Okay - I've added a short writeup under the "Commentary" section in 
matlab-cell.el. Let me know if this is what you had in mind.
> The keyb binding is something we might discuss later
>
> Thanks for the code
>
> Uwe
Thanks for taking a look!
Nidish