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

Bug 320285

Summary:

Messages inteneded for developers in the error log

Product:

[Eclipse Project] e4

Reporter:

Oleg Besedin <ob1.eclipse>

Component:

Assignee:

Project Inbox <e4.ui-inbox>

Status:

RESOLVED FIXED

QA Contact:

Severity:

normal

Priority:

CC:

bokowski, ob1.eclipse, remy.suen

Version:

1.0

Flags:

remy.suen: review+
bokowski: review+

Target Milestone:

1.0 RC3

Hardware:

All

OS:

All

Whiteboard:

Attachments:

Description	Flags
Patch	none
Updated patch	none

Description Oleg Besedin

2010-07-19 11:36:09 EDT

The message "unsupported: isEditorAreaVisible" appears in the error log during the normal run of e4 SDK. 

(For instance, starting the e4 SDK on an clean workspace produces two of those messages.)

Comment 1 Oleg Besedin

2010-07-23 10:05:19 EDT

*** Bug 320286 has been marked as a duplicate of this bug. ***

Comment 2 Oleg Besedin

2010-07-23 10:06:13 EDT

Changing title to reflect generic problem.

Comment 3 Oleg Besedin

2010-07-23 10:10:38 EDT

Created attachment 175066 [details]
Patch

Patch changes E4Util#unsupported() to use a new trace option from the org.eclipse.ui.workbench bundle:

  debug/unimplemented

Comment 4 Oleg Besedin

2010-07-23 10:14:42 EDT

I'll also do a sweep for logs & sysouts in this bug. For instance:

!ENTRY org.eclipse.ui 4 4 2010-07-23 10:04:59.328
!MESSAGE LegacyEvalContext.dispose: should it do something?

Comment 5 Oleg Besedin

2010-07-23 10:45:30 EDT

Created attachment 175070 [details]
Updated patch

Added several more cases that were producing console output; chnaged the tracing key to:

  debug/e4

Comment 6 Oleg Besedin

2010-07-23 10:46:14 EDT

Remy, Boris, coudl you review the patch?

Comment 7 Remy Suen

2010-07-23 11:02:36 EDT

Looks safe to me, +1.

Comment 8 Oleg Besedin

2010-07-23 11:41:21 EDT

Patch applied to CVS Head.