|
From: Tak O. <Tak...@am...> - 2005-08-31 17:16:26
|
Surendra, Thanks for revitalizing the project. Subdividing the file is a good idea. Compared to other typical elisp packages table.el is extraordinarily large. I think a good division can be where the next comment is located. ;; Worker functions (executed implicitly) In regard to the emacs CVS, the main difference of the code from sourceforge one is the departure from use of function advising. In the old package some functions are advised globally so that they can act specially when called in a table cell. Richard Stallman strongly discouraged using function advising in an elisp package that comes with emacs. He consider advising is for users to modify the default system behavior but not for the system implementation. The latest emacs has a new feature called "command remapping" and this is the mechanism I utilize in order to eliminate the use of function advise. Here is an excerpt from etc/NEWS. - Command remappings are defined using `define-key' with a prefix-key `remap', i.e. `(define-key MAP [remap CMD] DEF)' remaps command CMD to definition DEF in keymap MAP. The definition is not limited to another command; it can be anything accepted for a normal binding. Regards, -Tak Tue, 30 Aug 2005 23:09:07 -0700: <EFu...@ne...> wrote: > Hello, > I have created a cvs repository and checked in some changes to the 'table.el'. > There is also a Changes file which contains the changes which I have applied. > > > > Tak Ota <Tak...@am...> writes: > > I consider sourceforge version as generic table.el which runs on both > > emacs, xemacs and supports old emacsen too. The one in head of emacs > > CVS is the latest but does not run on old emacs or xemacs. Do you > > have a good idea how to harmonize or even somehow unify them? > > I have to look into the 'table.el' in the emacs CVS head to answer this question. > > Right now, I think the table.el file is too huge and hard to work with, I will try to refactor the code and move out some parts in other file. > > Cheers. > -- > Surendra Singhi > > http://www.public.asu.edu/~sksinghi/index.html > > > __________________________________________________________________ > Switch to Netscape Internet Service. > As low as $9.95 a month -- Sign up today at http://isp.netscape.com/register > > Netscape. Just the Net You Need. > > New! Netscape Toolbar for Internet Explorer > Search from anywhere on the Web and block those annoying pop-ups. > Download now at http://channels.netscape.com/ns/search/install.jsp |