Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge BETA_JAVA20 into master #209

Merged
merged 11 commits into from
Mar 23, 2023
Merged

Conversation

iloveeclipse
Copy link
Member

SarikaSinha and others added 11 commits January 27, 2023 17:22
eclipse-jdt#164

Change-Id: I325c71e0ac2b2f03c4ca0135fa0f04abab869e4d
Signed-off-by: Sarika Sinha <sarika.sinha@in.ibm.com>

Signed-off-by: Sarika Sinha <sarika.sinha@in.ibm.com>
* Catch and log exceptions from IJDIEventListener during event dispatch

IF IJDIEventListener.handleEvent() throws an exception during
EventDispatcher.dispatch(), the debugger breaks. Stepping, resuming,
terminating, etc. doesn't work.

This change adds a try-catch block around
IJDIEventListener.handleEvent() in EventDispatcher.dispatch(), to ensure
faulty listeners don't break the debugger.

Fixes: eclipse-jdt#172

* Disable GC for exception in ExceptionEvent until event is processed

This change disables garbage collection for
ExceptionEventImpl.fException, until the event is processed by listeners
in EventDispatcher.dispatch().

Fixes: eclipse-jdt#167

* Check if AskRecurrenceDialog was cancelled and has null result (eclipse-jdt#176)

Fixes eclipse-jdt#176

* Remove references to deprecated Navigator in Scrapbook editor

It's up for removal thus offering it makes no sense.

* Clean up duplicated code in JavaRuntime

Replace the code checking if a classpath entry is modular with the new
ClasspathEntry.isModular().

Fixes: eclipse-jdt#184

---------

Co-authored-by: Simeon Andreev <simeon.danailov.andreev@gmail.com>
Co-authored-by: Andrey Loskutov <loskutov@gmx.de>
Co-authored-by: Александър Куртаков <akurtakov@gmail.com>
* Catch and log exceptions from IJDIEventListener during event dispatch

IF IJDIEventListener.handleEvent() throws an exception during
EventDispatcher.dispatch(), the debugger breaks. Stepping, resuming,
terminating, etc. doesn't work.

This change adds a try-catch block around
IJDIEventListener.handleEvent() in EventDispatcher.dispatch(), to ensure
faulty listeners don't break the debugger.

Fixes: eclipse-jdt#172

* Disable GC for exception in ExceptionEvent until event is processed

This change disables garbage collection for
ExceptionEventImpl.fException, until the event is processed by listeners
in EventDispatcher.dispatch().

Fixes: eclipse-jdt#167

* Check if AskRecurrenceDialog was cancelled and has null result (eclipse-jdt#176)

Fixes eclipse-jdt#176

* Remove references to deprecated Navigator in Scrapbook editor

It's up for removal thus offering it makes no sense.

* Clean up duplicated code in JavaRuntime

Replace the code checking if a classpath entry is modular with the new
ClasspathEntry.isModular().

Fixes: eclipse-jdt#184

* POM and product version changes for 4.28 release

Fixes eclipse-platform/eclipse.platform.releng.aggregator#916

* Bump bundle version for 4.28 (eclipse-jdt#192)

See eclipse-jdt#192

* JDIType.throwDebugException() doesn't properly handle disconnected
exception (eclipse-jdt#192)

TL;DR: simply don't override the super method - it does already the
"right thing".

See longer explanation in eclipse-jdt#192.

Fixes eclipse-jdt#192

---------

Co-authored-by: Simeon Andreev <simeon.danailov.andreev@gmail.com>
Co-authored-by: Andrey Loskutov <loskutov@gmx.de>
Co-authored-by: Александър Куртаков <akurtakov@gmail.com>
Co-authored-by: Rahul Mohanan <rahul.mohanan@ibm.com>
* Refer to Y build for tests

* Updated Copyright
…pse-jdt#196)

unavailable --release 7

Co-authored-by: Sarika Sinha <sarika.sinha@in.ibm.com>
And also Update the Javadoc URL for JDK 20
Conflicts:
	org.eclipse.jdt.debug/META-INF/MANIFEST.MF
	org.eclipse.jdt.debug/pom.xml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants