matthiasblaesing commented on PR #9285: URL: https://github.com/apache/netbeans/pull/9285#issuecomment-4111931054
- I think the "don't break in lambda by default" is not a good approach. My reasoning: The feature is hard to discover, so without knowledge of this PR I would come to the conclusion "NetBeans can't break in lambdas". Breaking on the line outside the lambda (i.e. before execution of the chain) is IMHO the least interesting breakpoint as the intermediate steps are what I would be interested in. - The don't break by default setting is also harmful for people using multiple lines to make the lambda readable. Today they can set a break point and it will work as expected. With this the breakpoint you can set without hassle is basicly useless. I mean this formatting: <img width="513" height="222" alt="grafik" src="https://github.com/user-attachments/assets/2b9835e0-96be-4dac-853c-02a373fb475c"; /> - The lambda breakpoint enablement is unstable. Consider this starting point (nothing is running right now): <img width="1233" height="313" alt="grafik" src="https://github.com/user-attachments/assets/780a4be3-9486-41f8-a30f-feb291b4bd35"; /> When I invoke "Debug file", the activation of the lamdba breakpoint is lost (see lambda index 2): <img width="1246" height="346" alt="grafik" src="https://github.com/user-attachments/assets/a0e5acca-5513-4c5f-9429-4c7fc748cf1a"; /> What I wonder in summary is: Isn't this "just" another "condition"? The ability to break in lambda is already there, at least I can set a break point for the example in the row with multiple lambdas and get multiple stops, that match what I would expect for the lambda execution. What is missing from that POV is "just" a filter to select the right lambda(s). The drawback I see is, that the multiple breakpoints would allow more flexibility (independent counts for conditionally enabled breakpoints, conditions, log output). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] For further information about the NetBeans mailing lists, visit: https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
