https://bz.apache.org/bugzilla/show_bug.cgi?id=54330
Mark Thomas changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://issues.apache.org/bugzilla/show_bug.cgi?id=54330
--- Comment #4 from Filip Hanik ---
Patch looks good to me. I like the cleanup of using the interface.
--
You are receiving this mail because:
You are the assignee for the bug.
https://issues.apache.org/bugzilla/show_bug.cgi?id=54330
Christopher Schultz changed:
What|Removed |Added
Summary|Patch with some refactoring |Patch with some refactoring
https://issues.apache.org/bugzilla/show_bug.cgi?id=54330
Mark Thomas changed:
What|Removed |Added
Component|Catalina|Cluster
Hardware|Macintosh
https://issues.apache.org/bugzilla/show_bug.cgi?id=54330
Greg Turnquist changed:
What|Removed |Added
OS||All
--- Comment #2 from Greg Turn
https://issues.apache.org/bugzilla/show_bug.cgi?id=54330
Mark Thomas changed:
What|Removed |Added
Attachment #29779|0 |1
is patch|
https://issues.apache.org/bugzilla/show_bug.cgi?id=54330
Bug ID: 54330
Summary: Patch with some refactoring of Member.java
Product: Tomcat 8
Version: trunk
Hardware: Macintosh
Status: NEW
Severity: enhancement
Greg Turnquist wrote:
>Attached is a patch file where I went through looking for usage of
>concrete
>class MemberImpl, and when possible, replaced it with the interface
>Member.
>I added a few (maybe three) methods to the Member interface definition
>to
>support those changes. I ran the entire te
Attached is a patch file where I went through looking for usage of concrete
class MemberImpl, and when possible, replaced it with the interface Member.
I added a few (maybe three) methods to the Member interface definition to
support those changes. I ran the entire test suite, and the only tests th
