Bug 348277 - [api] Expose PatternConstructor
Summary: [api] Expose PatternConstructor
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Search (show other bugs)
Version: 3.7   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 3.8 M5   Edit
Assignee: Dani Megert CLA
QA Contact:
URL:
Whiteboard:
Keywords: api
Depends on:
Blocks: 310933
  Show dependency tree
 
Reported: 2011-06-03 16:31 EDT by Kevin Sawicki CLA
Modified: 2011-12-13 10:41 EST (History)
1 user (show)

See Also:

Attachments
Git Search dialog screenshot (83.42 KB, image/png)
2011-06-05 19:00 EDT, Kevin Sawicki CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Kevin Sawicki CLA 2011-06-03 16:31:25 EDT 
EGit 1.0 uses org.eclipse.search.internal.core.text.PatternConstructor.createPattern to create
patterns to match commit messages in the Git Search dialog. 

This class is internal and I haven't found a way to access it directly or indirectly without
getting a discouraged access violation.
Comment 1 Dani Megert CLA 2011-06-04 02:13:54 EDT 
>create patterns to match commit messages in the Git Search dialog.
Is that dialog part of the general search dialog? Can you attach a screenshot?
Comment 2 Kevin Sawicki CLA 2011-06-05 19:00:20 EDT 
Created attachment 197369 [details]
Git Search dialog screenshot
Comment 3 Dani Megert CLA 2011-06-06 02:38:38 EDT 
Which of the 'createPattern' methods do you need?
Comment 4 Kevin Sawicki CLA 2011-06-06 13:57:12 EDT 
We are using PatternConstructor.createPattern(String pattern, boolean isCaseSensitive, boolean isRegex)
Comment 5 Dani Megert CLA 2011-12-13 10:41:46 EST 
(In reply to comment #4)
> We are using PatternConstructor.createPattern(String pattern, boolean
> isCaseSensitive, boolean isRegex)

Fixed in master: ea87859a5d35cb349a6bc0de0278150a04f6bab2

See org.eclipse.search.core.text.TextSearchEngine.createPattern(String, boolean, boolean).

Please verify in one of the next builds whether this fits your needs.
Comment 6 Dani Megert CLA 2011-12-13 10:41:55 EST 
.