Skip to content

Fix Signature to check context before converting $ to dot#5064

Merged
jjohnstn merged 1 commit into
eclipse-jdt:masterfrom
jjohnstn:dollarfix
May 14, 2026
Merged

Fix Signature to check context before converting $ to dot#5064
jjohnstn merged 1 commit into
eclipse-jdt:masterfrom
jjohnstn:dollarfix

Conversation

@jjohnstn
Copy link
Copy Markdown
Contributor

What it does

See referenced issue.

How to test

See referenced issue.

Author checklist

- add a check in Signature.appendClassTypeSignature to see if a
  $ is preceded or followed by a dot before assuming it is an inner
  class and not part of a package name
- add new test to SignatureTests
- for eclipse-jdt#5056
@jjohnstn jjohnstn self-assigned this May 13, 2026
@jjohnstn jjohnstn added the bug Something isn't working label May 13, 2026
@jjohnstn jjohnstn added this to the 4.40 M3 milestone May 13, 2026
@jjohnstn jjohnstn merged commit f10a4e6 into eclipse-jdt:master May 14, 2026
13 checks passed
@jjohnstn jjohnstn deleted the dollarfix branch May 14, 2026 14:42
@cdietrich
Copy link
Copy Markdown

might this have been causing eclipse-xtext/xtext#3708

@cdietrich
Copy link
Copy Markdown

@jjohnstn this seems to break a typename like

org.eclipse.xtext.common.types.testSetups.$StartsWithDollar

package org.eclipse.xtext.common.types.testSetups;

public class $StartsWithDollar {

	public static class Builder {}
	
}

cc @iloveeclipse

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants