chen870647924 / guava-libraries

Automatically exported from code.google.com/p/guava-libraries
Apache License 2.0
0 stars 0 forks source link

toString for CharMatcher #882

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Hi,

 it would be nice if CharMatcher had a proper toString() method, so i can output it's filter-criteria in log-statements. Would that be complicated?

Regards

Original issue reported on code.google.com by fabian.z...@gmail.com on 23 Jan 2012 at 10:14

GoogleCodeExporter commented 9 years ago
Accepted, until/unless I find some reason this is difficult or impossible.

Original comment by wasserman.louis on 23 Jan 2012 at 5:20

GoogleCodeExporter commented 9 years ago
We have been trending toward providing toString() impls that somewhat mimic 
what the code used to create them looks like.  I see no big disadvantage to 
providing these for CharMatchers, I think we just never thought of it.

Original comment by kevinb@google.com on 25 Jan 2012 at 7:50

GoogleCodeExporter commented 9 years ago

Original comment by wasserman.louis on 25 Jan 2012 at 9:25

GoogleCodeExporter commented 9 years ago
Imported + working on the review internally now...thanks Louis!

Original comment by kak@google.com on 27 Jan 2012 at 7:55

GoogleCodeExporter commented 9 years ago

Original comment by wasserman.louis on 31 Jan 2012 at 11:53

GoogleCodeExporter commented 9 years ago

Original comment by kak@google.com on 31 Jan 2012 at 11:54

GoogleCodeExporter commented 9 years ago
I noticed the anyOf() two character case is still missing a #toString() 
implementation

See Issue 1080

Original comment by nick.talbot@gmail.com on 24 Jul 2012 at 1:20

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:14

GoogleCodeExporter commented 9 years ago

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