https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
Alex Giotis alex.gio...@gmail.com changed:
What|Removed |Added
Status|RESOLVED|CLOSED
--
https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
Vincent Hennebert vhenneb...@gmail.com changed:
What|Removed |Added
Status|NEW |RESOLVED
https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
--- Comment #23 from Vincent Hennebert vhenneb...@gmail.com 2012-03-22
16:54:42 UTC ---
Hi Alexios,
(In reply to comment #21)
Vincent, thanks for looking at it.
* Why use Double.doubleToLongBits in equals methods to compare doubles
https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
--- Comment #24 from Vincent Hennebert vhenneb...@gmail.com 2012-03-22
18:00:46 UTC ---
Patch applied in rev. 1303891:
http://svn.apache.org/viewvc?rev=1303891view=rev
Sorry for the delay about this, and thanks for your patience.
I
https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
--- Comment #25 from Alex Giotis alex.gio...@gmail.com 2012-03-22 19:20:08
UTC ---
Thanks for reviewing and applying this patch !
I am happy that the deadlock is gone. I had a 2nd look on the final changes and
for me this issue is now
https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
--- Comment #26 from Mehdi Houshmand med1...@gmail.com 2012-03-22 21:55:19
UTC ---
snip/
And despite that, they probably don't even bring adequate coverage. Many
fields
are set to a mock of some property (see e.g. EnumLengthTestCase).
https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
--- Comment #20 from Vincent Hennebert vhenneb...@gmail.com 2012-03-21
20:41:37 UTC ---
Thanks for your patch! I have a few questions following a quick review:
* Why declare the equals and hashCode methods on interfaces (Numeric,
https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
--- Comment #21 from Alex Giotis alex.gio...@gmail.com 2012-03-22 02:02:02
UTC ---
Vincent, thanks for looking at it.
* Why declare the equals and hashCode methods on interfaces (Numeric,
PercentBase)? They are defined on Object
https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
--- Comment #22 from Alex Giotis alex.gio...@gmail.com 2012-03-22 02:08:31
UTC ---
This patch should also resolve Bug 50703.
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this
https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
--- Comment #17 from Mehdi Houshmand med1...@gmail.com 2012-03-09 09:12:12
UTC ---
snip/
- I don't think we get added value by testing the hashCode() equals()
implementations. The tests don't protect from future changes in the tested
https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
--- Comment #18 from Mehdi Houshmand med1...@gmail.com 2012-03-09 09:15:10
UTC ---
snip
Why not just put that in a unit test?? None of this code is tested and a
little
mistake could have far-reaching ramifications.
By that I obviously
https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
Alex Giotis alex.gio...@gmail.com changed:
What|Removed |Added
Attachment #28412|0 |1
is
https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
Alexis Giotis alex.gio...@gmail.com changed:
What|Removed |Added
Attachment #27495|0 |1
is
https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
--- Comment #16 from Alex Giotis alex.gio...@gmail.com 2012-03-02 16:33:36
UTC ---
*** Bug 51625 has been marked as a duplicate of this bug. ***
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You
https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
--- Comment #14 from Alexis Giotis alex.gio...@gmail.com 2011-12-15 12:27:57
UTC ---
Hi Medhi,
Now that you are a commiter and mockito is used, is there a reason for not
applying this patch ?
--
Configure bugmail:
https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
--- Comment #13 from Alexis Giotis alex.gio...@gmail.com 2011-10-24 16:59:20
UTC ---
Now that there was a vote and Mockito has been accepted, what about applying
the patch ? Is there something holding it ?
--
Configure bugmail:
https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
--- Comment #12 from Simon Pepping spepp...@apache.org 2011-09-27 08:25:35
UTC ---
(In reply to comment #9)
Some of the tests added require Mockito, I tried to avoid using mocking as
much
as possible for the obvious reason that the
https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
--- Comment #10 from Alexis Giotis alex.gio...@gmail.com 2011-09-15 12:09:38
UTC ---
Medhi, thanks for checking fixing checkstyle issues. Using the Property.eq()
definitely makes the equals() methods easier to read. I used the eclipse
https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
--- Comment #11 from Mehdi Houshmand med1...@gmail.com 2011-09-15 13:01:18
UTC ---
Excellent, thanks for looking over it, I agree, that //TODO shouldn't have
gotten through.
Thanks again for taking the time to check through it.
--
https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
Mehdi Houshmand med1...@gmail.com changed:
What|Removed |Added
Attachment #27477|0 |1
is
https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
Vincent Hennebert vhenneb...@gmail.com changed:
What|Removed |Added
Summary|Deadlock in PropertyCache |[PATCH]
https://issues.apache.org/bugzilla/show_bug.cgi?id=46962
Alexis Giotis alex.gio...@gmail.com changed:
What|Removed |Added
Attachment #27358|0 |1
is
22 matches
Mail list logo