| Summary: | Warning created by checkstyle plug-in don't appear. | ||
|---|---|---|---|
| Product: | [Eclipse Project] Platform | Reporter: | Randy Hudson <hudsonr> |
| Component: | Text | Assignee: | Platform-Text-Inbox <platform-text-inbox> |
| Status: | CLOSED INVALID | QA Contact: | |
| Severity: | normal | ||
| Priority: | P3 | ||
| Version: | 3.1 | ||
| Target Milestone: | --- | ||
| Hardware: | PC | ||
| OS: | Windows XP | ||
| Whiteboard: | |||
|
Description
Randy Hudson
Please open/report a bug against checkstyle. If their investigation results in being an Eclipse SDK bug they will file a bug with more details. Have you ever tried creating warning markers for which the range included the line and its trailing line delimiter? I'm going to guess that this is causing the problem. You can add markers (e.g. bookmarks) that span exactly one line or several lines. Maybe their markers end just between CR LF. If you can provide more details or steps we can track this further. I think your suspicion might be correct. Would it be too difficult to hack up one of the builders, spell checker, or something, to create a marker at say line 30 which starts at the beginning of the line, and terminates in the middle of the delimiter? Adding markers that end between cr lf works. Adding markers that end between cr lf works. Using the latest version of checkstyle, squiggles appear, but they are not on the same line as the icon for the same annotation in the vertical ruler. It seems that the annotation has the wrong offsets but the right line number. Do annotations contain redundant information? i.e., can they contain an absolute offset, and also a line number which *should* agree with the offset? I assume that the checkstyle plug-in doesn't do anything with annotations but simply creates markers which do not check anything i.e. you can set offset 0, length 0 and line to 100. You have three choices: 1) file a bug against the checkstyle guys 2) debug yourself 3) attach the checkstyle plug-in here so that we can debug it oh, choice for is to drop it ;-) |