Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 169855

Summary:	Usage of field org.eclipse.jdt.internal.compiler.ast.AbstractMethodDeclaration.modifiersSourceStart
Product:	[Eclipse Project] JDT	Reporter:	Stephan Herrmann <stephan.herrmann>
Component:	Core	Assignee:	David Audel <david_audel>
Status:	CLOSED WONTFIX	QA Contact:
Severity:	minor
Priority:	P3
Version:	3.2
Target Milestone:	---
Hardware:	PC
OS:	Linux
Whiteboard:	stalebug

Description Stephan Herrmann

2007-01-08 11:39:05 EST

The field org.eclipse.jdt.internal.compiler.ast.AbstractMethodDeclaration.modifiersSourceStart
is only used by the DocumentElementParser.

If this is intended it should probably be documented in AbstractMethodDeclaration.
Otherwise I would suggest to let Parser set the field to the
same value as declarationSourceStart.
(Also applies to CompletionParser; haven't checked other AST-nodes for similar issue)

I implemented reporting for a modifier-related problem and was quite confused 
to find 0 as the modifiers start position.

Comment 1 Philipe Mulet

2007-01-09 04:26:05 EST

Indeed, this feels inconsistent.
Do we really need this field at all ?

Comment 2 Stephan Herrmann

2009-01-22 16:50:35 EST

(In reply to comment #1)

> Do we really need this field at all ? 

A little browsing shows: there is only one class that ever actually 
uses this value: org.eclipse.jdt.internal.core.jdom.DOMBuilder.
Mh, that one's deprecated since 2.0, but it's still there.

Comment 3 Eclipse Genie

2020-01-10 15:14:44 EST

This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.