search

eclipse-mat / mat

The Eclipse Memory Analyzer is a fast and feature-rich Java heap dump analyzer that helps you find memory leaks and reduce memory consumption.
https://eclipse.dev/mat/
Eclipse Public License 2.0
69 stars 13 forks source link

Changes for RAP/RCP single-sourced version of Memory Analyzer #16

Open eclipsewebmaster opened 5 months ago

eclipsewebmaster commented 5 months ago

| --- | --- | | Bugzilla Link | 335356 | | Status | NEW | | Importance | P3 enhancement | | Reported | Jan 25, 2011 13:04 EDT | | Modified | Nov 08, 2011 09:28 EDT | | Version | 1.1 | | Blocks | 262603 | | Reporter | Andrew Johnson |

Description

CQ 4723 approved the changes from bug 262603 comment 1 for use in the Memory Analyzer project.

The approved code will be attached to this bug.

eclipsewebmaster commented 5 months ago

By Andrew Johnson on Jan 25, 2011 13:10

Created attachment 187550 Patch to existing MAT projects to enable RAP

Approved by CQ 4723

:notepad_spiral: matrap.patch

eclipsewebmaster commented 5 months ago

By Andrew Johnson on Jan 25, 2011 13:13

Created attachment 187552 Additional plugin projects for MAT for web enablement using RAP

Approved by CQ 4723

Hi Andrew:

I've been able to complete full review as part of the triage stage.

A couple of small changes that should be made before the content is checked in:

  1. ActionFactoryFacadeImpl.java contains the following "About", "(c) Copyright Eclipse contributors and others 2000, 2009. All rights reserved.")

Copyright should not be assigned to Eclipse contributors (please note this information is not contained in the header but included quite a distance down in the file)

  1. ...

Otherwise, this CQ is approved!

Cheers, Sharon

:compression: matss-workspace-mat-compatibility2.zip

eclipsewebmaster commented 5 months ago

By Andrew Johnson on Jan 25, 2011 13:16

Created attachment 187553 Modify eclipse help system for use in RAP environment

Approved by CQ 4723

Hi Andrew:

I've been able to complete full review as part of the triage stage.

A couple of small changes that should be made before the content is checked in:

  1. ...

  2. The patch contains the word 'hack', you may wish to adjust:-)

Otherwise, this CQ is approved!

Cheers, Sharon

:notepad_spiral: eclipse_help.patch

eclipsewebmaster commented 5 months ago

By Andrew Johnson on Jan 25, 2011 13:17

Created attachment 187554 RAP compatibility, forwarding UI calls from MAT to RAP

Approved by CQ 4723

:compression: rap_compatibility3.zip