mbien commented on PR #7617: URL: https://github.com/apache/netbeans/pull/7617#issuecomment-2255774840
I have to wonder though how many don't realize that the debugger table has a `toString()` column? The formatters are likely something which needs a good look. I am pretty sure that the original reason for providing default formatters was so that a `toString()` on collections wouldn't explode or cause concurrency issues and also to show that users could do the same if they encounter objects with similar problems. The `Map.Entry` formatter is already problematic alone for the fact that it simply calls `toString()` on something which could be a collection too, _without_ using the dedicated formatter for it. We should probably consider to render the `toString()` column by default but delay items by using `[show]` and `[show all]` buttons or something similar. And then reconsider what needs to be a default formatter. -- 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
