bug#12298: 24.2; wrong font locking for < in comments

bug#12298: 24.2; wrong font locking for < in comments

[Leo]

[-- Attachment #1: Type: text/plain, Size: 169 bytes --]

GNU Emacs 24.2.1 (i386-apple-darwin12.1.0, Carbon Version 1.6.0 AppKit 1187)
 of 2012-08-27

Open the attached k.java file and see the 4 <'s fontified in warning
face.


[-- Attachment #2: k.java --]
[-- Type: text/plain, Size: 626 bytes --]

/**
 * Returns a new object with cropped image data. Implementations may keep a reference to the
 * original data rather than a copy. Only callable if isCropSupported() is true.
 *
 * @param left The left coordinate, 0 <= left < getWidth().
 * @param top The top coordinate, 0 <= top <= getHeight().
 * @param width The width of the rectangle to crop.
 * @param height The height of the rectangle to crop.
 * @return A cropped version of this object.
 */
public LuminanceSource crop(int left, int top, int width, int height) {
    throw new UnsupportedOperationException("This luminance source does not support cropping.");
}

[-- Attachment #3: cc.png --]
[-- Type: image/png, Size: 60861 bytes --]

bug#12298: 24.2; wrong font locking for < in comments

[Stefan Kangas]

found 12298 27.0.50
thanks

Leo <sdl.web@gmail.com> writes:

> GNU Emacs 24.2.1 (i386-apple-darwin12.1.0, Carbon Version 1.6.0 AppKit 1187)
>  of 2012-08-27
>
> Open the attached k.java file and see the 4 <'s fontified in warning
> face.
>
> /**
>  * Returns a new object with cropped image data. Implementations may keep a reference to the
>  * original data rather than a copy. Only callable if isCropSupported() is true.
>  *
>  * @param left The left coordinate, 0 <= left < getWidth().
>  * @param top The top coordinate, 0 <= top <= getHeight().
>  * @param width The width of the rectangle to crop.
>  * @param height The height of the rectangle to crop.
>  * @return A cropped version of this object.
>  */
> public LuminanceSource crop(int left, int top, int width, int height) {
>     throw new UnsupportedOperationException("This luminance source does not support cropping.");
> }

I can reproduce this on current master.

Best regards,
Stefan Kangas

bug#12298: 24.2; wrong font locking for < in comments

[Benjamin Riefenstahl]

This is Javadoc and the content of Javadoc is HTML plus additional
markup using keywords starting with "@", see
<https://www.oracle.com/technetwork/java/javase/documentation/index-137868.html#format>.
So it seems warning face is correct here?

bug#12298: 24.2; wrong font locking for < in comments

[Stefan Kangas]

Benjamin Riefenstahl <b.riefenstahl@turtle-trading.net> writes:

> This is Javadoc and the content of Javadoc is HTML plus additional
> markup using keywords starting with "@", see
> <https://www.oracle.com/technetwork/java/javase/documentation/index-137868.html#format>.
> So it seems warning face is correct here?

Ah, you're right.  To put the literal "<" character, one has to type
"{@literal <}", although it seems like "&lt;" will also work.  (Sorry
for the confusion; I haven't touched Java in a decade or more...)

If no one objects, I'll just close this as notabug.

Best regards,
Stefan Kangas

bug#12298: 24.2; wrong font locking for < in comments

[Stefan Kangas]

tags 12298 + notabug
close 12298
quit

Stefan Kangas <stefan@marxist.se> writes:

> > This is Javadoc and the content of Javadoc is HTML plus additional
> > markup using keywords starting with "@", see
> > <https://www.oracle.com/technetwork/java/javase/documentation/index-137868.html#format>.
> > So it seems warning face is correct here?
>
> Ah, you're right.  To put the literal "<" character, one has to type
> "{@literal <}", although it seems like "&lt;" will also work.  (Sorry
> for the confusion; I haven't touched Java in a decade or more...)
>
> If no one objects, I'll just close this as notabug.

No objections within 8 days.  Closing.

Best regards,
Stefan Kangas