Removing (modify-syntax-entry ?# "<")

Removing (modify-syntax-entry ?# "<")

[Leo]

Hello Carsten,

Setting a comment starter without a corresponding comment ender is
problematic and the # creeps in mysteriously under auto-fill. For
example, in my current running emacs, this happens almost certainly in
all org files that has # in their header. The only (temporary) solution
seems to reboot emacs (which is painful and disruptive).

If you try eval (forward-comment 1) at the beginning of an org file that
has some "#+..." it will move to the end of file (the whole file is
regarded as one single comment). So when auto-fill a long text, it will
find the common prefix to be #.

In addition, I don't think org mode has clear comment syntax or ideas on
what to do with it.

I can't see any gain from (modify-syntax-entry ?# "<") so I am proposing
removing it entirely and get rid of this mysterious and annoying bug
once and for all.

The attached patch may (though I think it is quite safe) cause some bugs
but those will be fixable unlike the one mentioned above.

Best wishes,

Leo


diff --git a/lisp/org/org.el b/lisp/org/org.el
index aae49fa..bb36ed8 100644
--- a/lisp/org/org.el
+++ b/lisp/org/org.el
@@ -4208,7 +4208,6 @@ The following commands are available:
     (org-set-tag-faces 'org-tag-faces org-tag-faces))
   ;; Calc embedded
   (org-set-local 'calc-embedded-open-mode "# ")
-  (modify-syntax-entry ?# "<")
   (modify-syntax-entry ?@ "w")
   (if org-startup-truncated (setq truncate-lines t))
   (org-set-local 'font-lock-unfontify-region-function
@@ -4237,7 +4236,7 @@ The following commands are available:
 		 'org-block-todo-from-checkboxes))
 
   ;; Comment characters
-;  (org-set-local 'comment-start "#") ;; FIXME: this breaks wrapping
+  (org-set-local 'comment-start "#")
   (org-set-local 'comment-padding " ")
 
   ;; Align options lines



-- 
CCL-USER> (if you fail to plan (plan to fail))

Re: Removing (modify-syntax-entry ?# "<")

[Carsten Dominik]

Hi Leo,

thanks for this.

I am off for a one-week vacation now - I will look at this patch when  
I com back.

- Carsten

On Jun 7, 2010, at 1:12 PM, Leo wrote:

> Hello Carsten,
>
> Setting a comment starter without a corresponding comment ender is
> problematic and the # creeps in mysteriously under auto-fill. For
> example, in my current running emacs, this happens almost certainly in
> all org files that has # in their header. The only (temporary)  
> solution
> seems to reboot emacs (which is painful and disruptive).
>
> If you try eval (forward-comment 1) at the beginning of an org file  
> that
> has some "#+..." it will move to the end of file (the whole file is
> regarded as one single comment). So when auto-fill a long text, it  
> will
> find the common prefix to be #.
>
> In addition, I don't think org mode has clear comment syntax or  
> ideas on
> what to do with it.
>
> I can't see any gain from (modify-syntax-entry ?# "<") so I am  
> proposing
> removing it entirely and get rid of this mysterious and annoying bug
> once and for all.
>
> The attached patch may (though I think it is quite safe) cause some  
> bugs
> but those will be fixable unlike the one mentioned above.
>
> Best wishes,
>
> Leo
>
>
> diff --git a/lisp/org/org.el b/lisp/org/org.el
> index aae49fa..bb36ed8 100644
> --- a/lisp/org/org.el
> +++ b/lisp/org/org.el
> @@ -4208,7 +4208,6 @@ The following commands are available:
>     (org-set-tag-faces 'org-tag-faces org-tag-faces))
>   ;; Calc embedded
>   (org-set-local 'calc-embedded-open-mode "# ")
> -  (modify-syntax-entry ?# "<")
>   (modify-syntax-entry ?@ "w")
>   (if org-startup-truncated (setq truncate-lines t))
>   (org-set-local 'font-lock-unfontify-region-function
> @@ -4237,7 +4236,7 @@ The following commands are available:
> 		 'org-block-todo-from-checkboxes))
>
>   ;; Comment characters
> -;  (org-set-local 'comment-start "#") ;; FIXME: this breaks wrapping
> +  (org-set-local 'comment-start "#")
>   (org-set-local 'comment-padding " ")
>
>   ;; Align options lines
>
>
>
> -- 
> CCL-USER> (if you fail to plan (plan to fail))
>
>
> _______________________________________________
> Emacs-orgmode mailing list
> Please use `Reply All' to send replies to the list.
> Emacs-orgmode@gnu.org
> http://lists.gnu.org/mailman/listinfo/emacs-orgmode

- Carsten

Re: Removing (modify-syntax-entry ?# "<")

[Carsten Dominik]

Hi Leo,

I have now applied this patch - hopefully it will do the right thing.
Thank you for your persistence in trying to fix this old and super- 
annoying
bug.

- Carsten

On Jun 7, 2010, at 1:12 PM, Leo wrote:

> Hello Carsten,
>
> Setting a comment starter without a corresponding comment ender is
> problematic and the # creeps in mysteriously under auto-fill. For
> example, in my current running emacs, this happens almost certainly in
> all org files that has # in their header. The only (temporary)  
> solution
> seems to reboot emacs (which is painful and disruptive).
>
> If you try eval (forward-comment 1) at the beginning of an org file  
> that
> has some "#+..." it will move to the end of file (the whole file is
> regarded as one single comment). So when auto-fill a long text, it  
> will
> find the common prefix to be #.
>
> In addition, I don't think org mode has clear comment syntax or  
> ideas on
> what to do with it.
>
> I can't see any gain from (modify-syntax-entry ?# "<") so I am  
> proposing
> removing it entirely and get rid of this mysterious and annoying bug
> once and for all.
>
> The attached patch may (though I think it is quite safe) cause some  
> bugs
> but those will be fixable unlike the one mentioned above.
>
> Best wishes,
>
> Leo
>
>
> diff --git a/lisp/org/org.el b/lisp/org/org.el
> index aae49fa..bb36ed8 100644
> --- a/lisp/org/org.el
> +++ b/lisp/org/org.el
> @@ -4208,7 +4208,6 @@ The following commands are available:
>     (org-set-tag-faces 'org-tag-faces org-tag-faces))
>   ;; Calc embedded
>   (org-set-local 'calc-embedded-open-mode "# ")
> -  (modify-syntax-entry ?# "<")
>   (modify-syntax-entry ?@ "w")
>   (if org-startup-truncated (setq truncate-lines t))
>   (org-set-local 'font-lock-unfontify-region-function
> @@ -4237,7 +4236,7 @@ The following commands are available:
> 		 'org-block-todo-from-checkboxes))
>
>   ;; Comment characters
> -;  (org-set-local 'comment-start "#") ;; FIXME: this breaks wrapping
> +  (org-set-local 'comment-start "#")
>   (org-set-local 'comment-padding " ")
>
>   ;; Align options lines
>
>
>
> -- 
> CCL-USER> (if you fail to plan (plan to fail))
>
>
> _______________________________________________
> Emacs-orgmode mailing list
> Please use `Reply All' to send replies to the list.
> Emacs-orgmode@gnu.org
> http://lists.gnu.org/mailman/listinfo/emacs-orgmode

- Carsten

Re: Removing (modify-syntax-entry ?# "<")

[Leo]

On 2010-06-18 08:11 +0100, Carsten Dominik wrote:
> Hi Leo,
>
> I have now applied this patch - hopefully it will do the right thing.
> Thank you for your persistence in trying to fix this old and super-
> annoying bug.
>
> - Carsten

Thank you Carsten.

I hope people who use org mode features more extensively can test it
out. My personal use since the patch was submitted indicates no problem
but my org mode setup is only ~50 lines.

Cheers,
Leo