This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.
Summary: | Need for matcher factory - a factory that creates matchers for the given SearchTypes | ||
---|---|---|---|
Product: | utilities | Reporter: | Vladimir Kvashin <vkvashin> |
Component: | Jump To | Assignee: | Tomas Zezula <tzezula> |
Status: | VERIFIED FIXED | ||
Severity: | normal | CC: | dstrupl, issues, vvg |
Priority: | P3 | Keywords: | API, API_REVIEW_FAST |
Version: | 6.x | ||
Hardware: | All | ||
OS: | All | ||
Issue Type: | DEFECT | Exception Reporter: | |
Attachments: | Patch |
Description
Vladimir Kvashin
2010-04-28 10:37:49 UTC
Tomas, I've found your activities this month in the area of the org.netbeans.spi.jumpto : - Bug 182884 - http://hg.netbeans.org/main/rev/9a59d1812167 Could you please, validate Vladimir's proposal. Hi Victor, the Vladimir's classes looks good. They should belong to friend support SPI in the jump to. Should I do it? Or are you going to do this? Tomas, I think, you more closer to this theme now, and you'll able to provide improvement of code on the fly if such changes will be needed. Could you please, do it. OK, I will do it. The cnd will need to be updated to use it. It will also need an API review. I will add the link to the review when done. Could you or Vladimir do then the review? Thanks. Tomas, candidates to migration on support api could be: spring.beans/src/org/netbeans/modules/spring/beans/jumpto/SpringBeansTypeProvider.java java.sourceui/src/org/netbeans/modules/java/source/ui/JavaSymbolProvider.java gsf/src/org/netbeans/modules/gsf/GsfTypeProvider.java cnd.gotodeclaration/src/org/netbeans/modules/cnd/gotodeclaration/matcher/NameMatcherFactory.java OK, thanks Vladimir. Tomas, yes, I can do the review I've moved the NameMatcherFactory and NameMatcher into Jump To. It's basically the same. I've only removed the cnd specific method createNameFilter() and changed the createNameMatcher() to throw IAE when it gets wrong type or PatternSyntaxException as it's an exceptional case which should be reported and fixed. The rest is the same. I've also changed cnd, spring.beans, GoTo File action to use it. For cnd there is the original NameMatcherFactory having the single method createNameFilter. I would rather rename it but I don't know a suitable name. Can you suggest one? Thanks. Created attachment 98352 [details]
Patch
As it's just a friend API I will integrate it tomorrow. Sure I agree with your changes :) OK, thanks Vladimir. I will integrate it tomorrow. Integrated into jet-main 7c2c70892ef1 jet-main 722eca07970c - removed forgotten NameMatcher. Integrated into 'main-golden', will be available in build *201005100200* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress) Changeset: http://hg.netbeans.org/main/rev/ User: Log: |