Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 405316

Summary: replace String.length() == 0 with String.isEmpty()
Product: z_Archived Reporter: Miles Parker <milesparker>
Component: MylynAssignee: Mylyn Inbox <mylyn-inbox>
Status: RESOLVED WONTFIX QA Contact:
Severity: enhancement    
Priority: P3 CC: steffen.pingel
Version: unspecified   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Bug Depends on:    
Bug Blocks: 383907    

Description Miles Parker CLA 2013-04-09 15:30:41 EDT 
This one is an aesthetic nit, but I'd love to fix all of these now that we're on Java 1.6. Happy to put together the reviews for this...
Comment 1 Steffen Pingel CLA 2013-04-10 09:50:17 EDT 
Unless there is a concrete benefit other than personal preference I wouldn't want to invest the effort. It's fine with me to use isEmpty() in new code that is being or written or update existing code as we touch it. We can discuss further during today's call but I'm inclined mark this as won't fix.
Comment 2 Steffen Pingel CLA 2013-04-15 06:07:32 EDT 
Closing as per discussion at the last weekly meeting. If anyone has additional thought please feel free to re-open.