Giving anonymous inner classes a name

kucci / guava-libraries

Automatically exported from code.google.com/p/guava-libraries

Apache License 2.0

0 stars 0 forks source link

Giving anonymous inner classes a name #390

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago


For instance Iterators.concat returns a value of type Iterators$5

This is not very meaningful when visible in stacktraces or during step-through 
debugging

the code

return new Iterator<T>() { ... }

could be replaced with

class ConcatIterator implements Iterator<T> { ... }
return new ConcatIterator()

As far as I know, the difference is entirely cosmetic. I'll agree it's a very 
tedious thing to do, but it really helps debugging.

Original issue reported on code.google.com by jvdne...@gmail.com on 30 Jul 2010 at 10:45

GoogleCodeExporter commented 9 years ago

We'll consider it... but we'd likely be very tempted to change them right back 
again once lambdas arrive in the language.

Original comment by kevinb@google.com on 30 Jul 2010 at 3:21

GoogleCodeExporter commented 9 years ago

Original comment by fry@google.com on 28 Jan 2011 at 4:55

Changed state: Accepted
Added labels: Type-Enhancement

GoogleCodeExporter commented 9 years ago

Original comment by kevinb@google.com on 13 Jul 2011 at 6:18

Changed state: Triaged

GoogleCodeExporter commented 9 years ago

Original comment by fry@google.com on 16 Feb 2012 at 7:17

Changed state: Acknowledged

GoogleCodeExporter commented 9 years ago

Original comment by kevinb@google.com on 2 Mar 2012 at 6:51

Added labels: Package-General

GoogleCodeExporter commented 9 years ago

Original comment by kevinb@google.com on 30 May 2012 at 7:41

Added labels: Type-Enhancement-Temp
Removed labels: Type-Enhancement

GoogleCodeExporter commented 9 years ago

Original comment by kevinb@google.com on 30 May 2012 at 7:45

Added labels: Type-Enhancement
Removed labels: Type-Enhancement-Temp

GoogleCodeExporter commented 9 years ago

Original comment by kevinb@google.com on 22 Jun 2012 at 6:16

Changed state: Research

GoogleCodeExporter commented 9 years ago

This issue has been migrated to GitHub.

It can be found at https://github.com/google/guava/issues/<id>

Original comment by cgdecker@google.com on 1 Nov 2014 at 4:15

Added labels: MigratedToGitHub

GoogleCodeExporter commented 9 years ago

Original comment by cgdecker@google.com on 1 Nov 2014 at 4:18

Changed state: Migrated

GoogleCodeExporter commented 9 years ago

Original comment by cgdecker@google.com on 3 Nov 2014 at 9:09

Added labels: Restrict-AddIssueComment-Commit