Bug 563208 - Toggle method should work on a list of methods containing abstract method
Summary: Toggle method should work on a list of methods containing abstract method
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Debug (show other bugs)
Version: 4.16   Edit
Hardware: PC Linux
: P3 enhancement (vote)
Target Milestone: 4.19 M1   Edit
Assignee: Vikas Chandra CLA
QA Contact:
URL:
Whiteboard:
Keywords: noteworthy
Depends on:
Blocks: 570008
  Show dependency tree
 
Reported: 2020-05-15 05:49 EDT by Vikas Chandra CLA
Modified: 2021-01-06 05:12 EST (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Vikas Chandra CLA 2020-05-15 05:49:22 EDT
Steps
1)Open AssistParser.java
2) Add an abstract method consumeMyMethod
@Override
protected void consumeClassBodyopt() {
	super.consumeClassBodyopt();
	popElement(K_SELECTOR);
}

abstract public void consumeMyMethod() ;
@Override
protected void consumeClassHeader() {
	super.consumeClassHeader();
	pushOnElementStack(K_TYPE_DELIMITER);
}

3) Now in outline view try to select all consume methods and put a breakpoint.

Addition of 1 abstract method doesn't allow this.

Behaviour should be similar to close projects which when selected on a mix of closed and open projects does nothing to the closed projects but closes open projects in the list.
Comment 1 Eclipse Genie CLA 2020-12-14 10:23:11 EST
New Gerrit change created: https://git.eclipse.org/r/c/jdt/eclipse.jdt.debug/+/173753
Comment 2 Sarika Sinha CLA 2020-12-16 07:56:42 EST
This was c hanged as part of Bug 112613, It was supposed to do for "remote" but don't see any reason to block for abstract methods.
Comment 4 Sarika Sinha CLA 2020-12-21 03:32:47 EST
As this is a behaviour change, We should add an entry in N&N.

Thanks Vikas for working on this.
Comment 5 Sarika Sinha CLA 2021-01-01 05:42:53 EST
Eclipse SDK

Version: 2021-03 (4.19)
Build id: I20201231-1800
Comment 6 Eclipse Genie CLA 2021-01-06 05:00:56 EST
New Gerrit change created: https://git.eclipse.org/r/c/www.eclipse.org/eclipse/news/+/174311