Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 331958 - API Use Scan problem description can get too long
Summary: API Use Scan problem description can get too long
Status: VERIFIED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: API Tools (show other bugs)
Version: 3.7   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 3.7 M5   Edit
Assignee: Curtis Windatt CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-12-06 16:51 EST by Curtis Windatt CLA
Modified: 2010-12-13 14:14 EST (History)
1 user (show)

See Also:
ankur_sharma: review+


Attachments
Fix (2.91 KB, patch)
2010-12-10 17:35 EST, Curtis Windatt CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Curtis Windatt CLA 2010-12-06 16:51:56 EST
I deleted a reference that is used by many consumers in my use scan.  The problem marker showed up correctly, but has many repeated plug-in entries.  Having the same plug-in listed more than once is not helpful.  Furthermore, the description text limit is hit, so it doesn't tell me what the problem actually is.

Three suggestions

1) Remove duplicate plug-ins
2) If the text is too long, replace with a ... and others
3) Put the basic description at the start so it can be understood from the problems view.
Comment 1 Curtis Windatt CLA 2010-12-06 16:57:34 EST
Description	Resource	Path	Location	Type
The method getSharedImages in type org.eclipse.jdt.ui.JavaUI called by org.eclipse.jdt.debug.ui, org.eclipse.jdt.junit, org.eclipse.jdt.junit, org.eclipse.jdt.junit, org.eclipse.jdt.junit, org.eclipse.jdt.junit, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.api.tools.ui, org.eclipse.pde.ds.ui, org.eclipse.pde.ui, org.eclipse.pde.ui, org.eclipse.pde.ui, org.eclipse.pde.ui, org.eclipse.pde.ui, org.eclipse.pde.ui, org.eclipse.pde.ui, org.eclipse.pde.ui, org.eclipse.pde.ui, org.eclipse.pde.ui, org.eclipse.pde.ui, org.eclipse.pde.ui, org.eclipse.pde.ui, org.eclipse.pde.ui, org.eclipse.pde.ui has been deleted.	JavaUI.java	/org.eclipse.jdt.ui/ui/org/eclipse/jdt/ui	line 78	API Use Scan Problem
Comment 2 Curtis Windatt CLA 2010-12-10 17:35:23 EST
Created attachment 185008 [details]
Fix

Changes the problem messages to put the shorter strings first.  Fixes the duplication problem (incorrect string.split()).
Comment 3 Curtis Windatt CLA 2010-12-13 12:39:57 EST
Fixed in HEAD.

Ankur, please verify.
Comment 4 Ankur Sharma CLA 2010-12-13 13:45:06 EST
+1

i wish bugzilla sent a mail when added to review
Comment 5 Curtis Windatt CLA 2010-12-13 14:14:01 EST
(In reply to comment #4)
> +1
> 
> i wish bugzilla sent a mail when added to review

You should get an email with something like the following:

Michael Rennie <Michael_Rennie@ca.ibm.com> has asked Curtis Windatt
<cwindatt@ca.ibm.com> for review:
Bug 331863: API Tools test case failures in N20101204-2000

If you don't check your bugzilla preferences.