fontification question

fontification question

[JRancier]

[-- Attachment #1.1: Type: text/plain, Size: 560 bytes --]

Given the following code:

(setq font-lock-todo-face (make-face 'font-lock-todo-face))
(set-face-foreground 'font-lock-todo-face "Red")
(set-face-background 'font-lock-todo-face "Yellow")

(font-lock-add-keywords 'jde-mode
                        '(("\\(TODO:.*$\\)" 1 font-lock-todo-face 
prepend)))

Can anyone tell me why the following is correctly fontified (in jde-mode):

    /*
     *
     *  TODO: REMOVE DEBUG STATEMENTS 
     * 
     */

And this isn't?

    /** 01-17-08  jbr
     *
     *  TODO: REMOVE DEBUG STATEMENTS 
     * 
     */

TIA,
Jeff

[-- Attachment #1.2: Type: text/html, Size: 1697 bytes --]

[-- Attachment #2: Type: text/plain, Size: 152 bytes --]

_______________________________________________
help-gnu-emacs mailing list
help-gnu-emacs@gnu.org
http://lists.gnu.org/mailman/listinfo/help-gnu-emacs

Re: fontification question

[Lennart Borgman (gmail)]

JRancier@penntraffic.com wrote:
> 
> Given the following code:
> 
> (setq font-lock-todo-face (make-face 'font-lock-todo-face))
> (set-face-foreground 'font-lock-todo-face "Red")
> (set-face-background 'font-lock-todo-face "Yellow")
> 
> (font-lock-add-keywords 'jde-mode
>                         '(("\\(TODO:.*$\\)" 1 font-lock-todo-face 
> prepend)))
> 
> Can anyone tell me why the following is correctly fontified (in jde-mode):
> 
>     /*
>      *
>      *  TODO: REMOVE DEBUG STATEMENTS
>      *
>      */
> 
> And this isn't?
> 
>     /** 01-17-08  jbr
>      *
>      *  TODO: REMOVE DEBUG STATEMENTS
>      *
>      */
> 
> TIA,
> Jeff


I am surprised that the first one works. Should it really do that?

Perhaps you can use hi-lock-mode?

Re: fontification question

[JRancier]

[-- Attachment #1.1: Type: text/plain, Size: 1064 bytes --]

"Lennart Borgman (gmail)" <lennart.borgman@gmail.com> wrote on 01/17/2008 
05:51:49 PM:

| JRancier@penntraffic.com wrote:
| > 
| > Given the following code:
| > 
| > (setq font-lock-todo-face (make-face 'font-lock-todo-face))
| > (set-face-foreground 'font-lock-todo-face "Red")
| > (set-face-background 'font-lock-todo-face "Yellow")
| > 
| > (font-lock-add-keywords 'jde-mode
| >                         '(("\\(TODO:.*$\\)" 1 font-lock-todo-face 
| > prepend)))
| > 
| > Can anyone tell me why the following is correctly fontified (in 
jde-mode):
| > 
| >     /*
| >      *
| >      *  TODO: REMOVE DEBUG STATEMENTS
| >      *
| >      */
| > 
| > And this isn't?
| > 
| >     /** 01-17-08  jbr
| >      *
| >      *  TODO: REMOVE DEBUG STATEMENTS
| >      *
| >      */
| > 
| > TIA,
| > Jeff
| 
| 
| I am surprised that the first one works. Should it really do that?

It does.  Can you clue me into why is 'shouldn't' work?  The interesting 
thing is that in lisp-mode and c-mode, it seems to always work fine ... 
???

| 
| Perhaps you can use hi-lock-mode?

[-- Attachment #1.2: Type: text/html, Size: 1670 bytes --]

[-- Attachment #2: Type: text/plain, Size: 152 bytes --]

_______________________________________________
help-gnu-emacs mailing list
help-gnu-emacs@gnu.org
http://lists.gnu.org/mailman/listinfo/help-gnu-emacs

Re: fontification question

[JRancier]

[-- Attachment #1.1: Type: text/plain, Size: 1525 bytes --]

"Lennart Borgman (gmail)" <lennart.borgman@gmail.com> wrote on 01/18/2008 
02:35:16 PM:

| JRancier@penntraffic.com wrote:
| > 
| > "Lennart Borgman (gmail)" <lennart.borgman@gmail.com> wrote on 
| > 01/17/2008 05:51:49 PM:
| > 
| > | JRancier@penntraffic.com wrote:
| > | >
| > | > Given the following code:
| > | >
| > | > (setq font-lock-todo-face (make-face 'font-lock-todo-face))
| > | > (set-face-foreground 'font-lock-todo-face "Red")
| > | > (set-face-background 'font-lock-todo-face "Yellow")
| > | >
| > | > (font-lock-add-keywords 'jde-mode
| > | >                         '(("\\(TODO:.*$\\)" 1 font-lock-todo-face
| > | > prepend)))
| > | >
| > | > Can anyone tell me why the following is correctly fontified (in 
| > jde-mode):
| > | >
| > | >     /*
| > | >      *
| > | >      *  TODO: REMOVE DEBUG STATEMENTS
| > | >      *
| > | >      */
| > | >
| > | > And this isn't?
| > | >
| > | >     /** 01-17-08  jbr
| > | >      *
| > | >      *  TODO: REMOVE DEBUG STATEMENTS
| > | >      *
| > | >      */
| > | >
| > | > TIA,
| > | > Jeff
| > |
| > |
| > | I am surprised that the first one works. Should it really do that?
| > 
| > It does.  Can you clue me into why is 'shouldn't' work?  The 
interesting 
| > thing is that in lisp-mode and c-mode, it seems to always work fine 
... ???
| 
| 
| I must have done something stupid, please ignore my answer.

That's quite alright, you have any idea why one works and the other 
doesn't?  After playing around a bit, the two '*' in a row seem to cause 
it to fail?

[-- Attachment #1.2: Type: text/html, Size: 2277 bytes --]

[-- Attachment #2: Type: text/plain, Size: 152 bytes --]

_______________________________________________
help-gnu-emacs mailing list
help-gnu-emacs@gnu.org
http://lists.gnu.org/mailman/listinfo/help-gnu-emacs

Re: fontification question

[Lennart Borgman (gmail)]

> That's quite alright, you have any idea why one works and the other 
> doesn't?  After playing around a bit, the two '*' in a row seem to cause 
> it to fail?

Fontification is quite tricky, especially the comments. It looks like a 
bug too me. If you are using a recent Emacs then you should perhaps file 
a bug report?

Re: fontification question

[JRancier]

[-- Attachment #1.1: Type: text/plain, Size: 803 bytes --]

"Lennart Borgman (gmail)" <lennart.borgman@gmail.com> wrote on 01/18/2008 
03:16:02 PM:

| > That's quite alright, you have any idea why one works and the other 
| > doesn't?  After playing around a bit, the two '*' in a row seem to 
cause 
| > it to fail?
| 
| Fontification is quite tricky, especially the comments. It looks like a 
| bug too me. If you are using a recent Emacs then you should perhaps file 

| a bug report?

Not a problem, as long as someone can verify my syntax is OK.  Here's the 
snippet again:

(setq font-lock-todo-face (make-face 'font-lock-todo-face))
(set-face-foreground 'font-lock-todo-face "Red")
(set-face-background 'font-lock-todo-face "Yellow")

(font-lock-add-keywords 'jde-mode
                        '(("\\<\\(TODO:.*$\\)" 1 font-lock-todo-face 
prepend)))


Jeff

[-- Attachment #1.2: Type: text/html, Size: 1243 bytes --]

[-- Attachment #2: Type: text/plain, Size: 152 bytes --]

_______________________________________________
help-gnu-emacs mailing list
help-gnu-emacs@gnu.org
http://lists.gnu.org/mailman/listinfo/help-gnu-emacs