This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 320285 - Messages inteneded for developers in the error log
Summary: Messages inteneded for developers in the error log
Status: RESOLVED FIXED
Alias: None
Product: e4
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 1.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.0 RC3   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 320286 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-07-19 11:36 EDT by Oleg Besedin CLA
Modified: 2010-07-23 11:41 EDT (History)
3 users (show)

See Also:
remy.suen: review+
bokowski: review+

Attachments
Patch (4.54 KB, patch)
2010-07-23 10:10 EDT, Oleg Besedin CLA 		no flags Details | Diff
Updated patch (11.82 KB, patch)
2010-07-23 10:45 EDT, Oleg Besedin CLA 		no flags Details | Diff
Show Obsolete (1) 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 Oleg Besedin CLA 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 CLA 2010-07-23 10:05:19 EDT 
*** Bug 320286 has been marked as a duplicate of this bug. ***
Comment 2 Oleg Besedin CLA 2010-07-23 10:06:13 EDT 
Changing title to reflect generic problem.
Comment 3 Oleg Besedin CLA 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 CLA 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 CLA 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 CLA 2010-07-23 10:46:14 EDT 
Remy, Boris, coudl you review the patch?
Comment 7 Remy Suen CLA 2010-07-23 11:02:36 EDT 
Looks safe to me, +1.
Comment 8 Oleg Besedin CLA 2010-07-23 11:41:21 EDT 
Patch applied to CVS Head.