From mboxrd@z Thu Jan 1 00:00:00 1970 From: chgreg692000@yahoo.com Subject: Re: org-mode + xemacs + patch Date: Thu, 11 May 2006 11:18:37 +0300 Message-ID: <7upsilvusi.fsf@horizonsemi.com> References: <7uejz3tgzd.fsf@horizonsemi.com> <3e5c1fd8f70c4ab0a1c887201cd1666f@science.uva.nl> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1Fe6Nq-0005vT-Vd for emacs-orgmode@gnu.org; Thu, 11 May 2006 04:18:55 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1Fe6Np-0005vH-Ho for emacs-orgmode@gnu.org; Thu, 11 May 2006 04:18:54 -0400 Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Fe6Np-0005vE-Fa for emacs-orgmode@gnu.org; Thu, 11 May 2006 04:18:53 -0400 Received: from [212.25.79.210] (helo=localhost.localdomain) by monty-python.gnu.org with esmtps (TLS-1.0:DHE_RSA_AES_256_CBC_SHA:32) (Exim 4.52) id 1Fe6PI-000473-8K for emacs-orgmode@gnu.org; Thu, 11 May 2006 04:20:24 -0400 In-Reply-To: <3e5c1fd8f70c4ab0a1c887201cd1666f@science.uva.nl> (Carsten Dominik's message of "Thu, 11 May 2006 05:58:19 +0200") List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org To: Carsten Dominik Cc: emacs-orgmode@gnu.org On Thu, 11 May 2006, dominik@science.uva.nl wrote: > Hi Greg, I have looked at you patch and taken most of it. However, > I have a few questions: > >> \\{org-mode-map}" >> + >> + ;; Get rid of Outline menus, they are not needed >> + ;; Need to do this here because define-derived-mode sets up >> + ;; the keymap so late. >> + (if (featurep 'xemacs) >> + (progn >> + (if org-noutline-p >> + (progn >> + (easy-menu-remove outline-mode-menu-heading) >> + (easy-menu-remove outline-mode-menu-show) >> + (easy-menu-remove outline-mode-menu-hide)) > > This assumes that the future port of noutline will use easy-menu for > its stuff. OK if you get your version into XEmacs, problematic > otherwise. I have taken it anyway, because right now it does not > break anything. > >> (get-char-property (1- (point)) 'invisible)) >> (beginning-of-line 2)) (setq eol (point))) >> (outline-end-of-heading) (setq eoh (point)) >> - (org-end-of-subtree t) (setq eos (point)) >> + ;;(org-end-of-subtree t) (setq eos (point)) >> + (outline-end-of-subtree) (setq eos (point)) >> (outline-next-heading)) > > Why is the above change necessary? Without this change 'org-cycle' gives error when 'org-cycle-include-plain-lists' is t. > >> (if (equal x "") (setq x "0")) >> - (setq form (replace-match (concat "(" x ")") t t form))) >> + ;;(setq form (replace-match (concat "(" x ")") t t form))) >> + (setq form (replace-match x t t form))) > > I guess you need this to get the value of variables directly into > the list form. I need the parenthesis, because fields in tables can > be complex expressions, and in an equation precedence rule may pull > this expression apart, which is not wanted. We need to do this in a > way that the parenthesis are only ignored for lisp expressions. > >> I've suspected that you added these parenthesis for some reason, but because my simple tests worked without them and because parenthesis used only in row formulas and not used in column formulas i removed them. I suppose it could be easy to add some if statement that will ignore parenthesis for elisp only. >> +(when (featurep 'xemacs) >> + (define-key org-mode-map 'button3 'popup-mode-menu)) >> + >> (defsubst org-table-p () (org-at-table-p)) > > > Any other XEmacs users who object to this change? > >> @@ -13772,7 +13790,8 @@ >> Only visible heading lines are considered, unless INVISIBLE-OK is >> non-nil." >> (if org-noutline-p >> (outline-back-to-heading invisible-ok) >> - (if (and (memq (char-before) '(?\n ?\r)) >> + (if (and (or (memq (char-before) '(?\n ?\r)) >> + (point-min)) >> (looking-at outline-regexp)) >> t > > This one I don't understand. Maybe you mean > > (if (and (or (bobp) (memq (char-before) '(?\n ?\r))) > > ???? Point-min is never nil, it is always just a number. > This is a bug. It should be (bobp). Have no idea why it's worked with (point-min). > > > Thanks. > > - Carsten Regards, Greg