Bug 531868 - SourceTypeBinding creates strange unique key on Windows
Summary: SourceTypeBinding creates strange unique key on Windows
Status: RESOLVED WORKSFORME
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 4.7.2   Edit
Hardware: PC Mac OS X
: P3 normal (vote)
Target Milestone: 4.21 M3   Edit
Assignee: Sravan Kumar Lakkimsetti CLA
QA Contact: Manoj N Palat CLA
URL:
Whiteboard:
Keywords: helpwanted
Depends on:
Blocks:
 
Reported: 2018-03-01 11:46 EST by Martin Lippert CLA
Modified: 2021-05-11 01:23 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Lippert CLA 2018-03-01 11:46:19 EST
SourceTypeBinding.computeUniqueKey(..)

creates unique keys for bindings, but the implementation doesn't work in the same way when running on Windows. For macOS (and Linux, I think), the unique keys look like (e.g.):

"Lorg/test/PersonHandler;"

for a declaring class (e.g.)

On windows, the same unique key for the same declaring class looks like:

Lorg/test/Z:\sts4\headless-services\spring-boot-language-server\target\test-classes\test-projects\test-webflux-project\src\main\java\org\test\PersonHandler~PersonHandler;

The reason for that is:

SourceTypeBinding assumes that the file name is always separated via "/" and not via "File.separator"
Comment 1 Eclipse Genie CLA 2020-03-02 08:40:39 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. As such, we're closing this bug.

If you have further information on the current state of the bug, please add it and reopen this bug. 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.
Comment 2 Martin Lippert CLA 2020-03-02 09:11:53 EST
Not sure if this is still an issue, but it would be worth to double check, I think
Comment 3 Manoj N Palat CLA 2020-03-03 01:44:25 EST
(In reply to Martin Lippert from comment #2)
> Not sure if this is still an issue, but it would be worth to double check, I
> think

Reopening - putting in the radar for 4.16
Comment 4 Till Brychcy CLA 2020-03-03 07:07:33 EST
.
Comment 5 Manoj N Palat CLA 2020-05-14 08:06:02 EDT
Bulk move out of 4.16
Comment 6 Manoj N Palat CLA 2020-10-01 05:02:35 EDT
Bulk move out of 4.18 M1
Comment 7 Manoj N Palat CLA 2021-02-19 07:25:59 EST
Bulk move out of 4.19
Comment 8 Sravan Kumar Lakkimsetti CLA 2021-05-11 01:23:51 EDT
I tried to reproduce this in multiple ways but unable to do so.

closing this as works for me. please reopen if you can reproduce this problem again.