Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 331699

Summary:

EditorsView's EditorsContentProvider can be replaced by ArrayContentProvider

Product:

[Eclipse Project] Platform

Reporter:

Remy Suen <remy.suen>

Component:

CVS

Assignee:

Tomasz Zarna <tomasz.zarna>

Status:

RESOLVED FIXED

QA Contact:

Severity:

minor

Priority:

Version:

3.7

Target Milestone:

3.7 M5

Hardware:

All

OS:

All

Whiteboard:

Attachments:

Description	Flags
Fix v01	none
mylyn/context/zip	none

Description Remy Suen

2010-12-02 12:20:09 EST

The content provider just returns the array as-is. The ArrayContentProvider from JFace can be used instead, see ArrayContentProvider.getInstance().

Comment 1 Tomasz Zarna

2010-12-03 11:49:06 EST

Yup, makes sense to me. How do you find these things Remy? ;)

Comment 2 Tomasz Zarna

2010-12-03 11:49:25 EST

Created attachment 184479 [details]
Fix v01

Comment 3 Tomasz Zarna

2010-12-03 11:49:27 EST

Created attachment 184480 [details]
mylyn/context/zip

Comment 4 Remy Suen

2010-12-03 12:01:44 EST

(In reply to comment #1)
> How do you find these things Remy? ;)

When I open random views for testing 4.x, I sometimes like to check the implementation for suspicious coding patterns that may cause problems in the future (even if the view opens up properly "today").

Comment 5 Tomasz Zarna

2010-12-14 05:31:33 EST

Fixed in HEAD, available in builds >=I20101214-0800. Thanks Remy.