Patches item #2877961, was opened at 2009-10-13 11:46
Message generated for change (Settings changed) made by rajarshi
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2877961&group_id=20024Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: Needs Revision
>Status: Closed
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Egon Willighagen (egonw)
Assigned to: Rajarshi Guha (rajarshi)
Summary: Extraction of common unit tests for IFingerprinter
Initial Comment:
This patch aimed at master, refactors the unit tests for IFingerprinter implementations, providing the unit tests for a few bugs to all implementations.
Rajarshi, you are most familiar with the IFingerprinter code; can you (as a minimum) review the two patches?
----------------------------------------------------------------------
Comment By: Rajarshi Guha (rajarshi)
Date: 2009-10-13 20:15
Message:
Patches look good. Applied to master and pushed to GitHub
----------------------------------------------------------------------
Comment By: Egon Willighagen (egonw)
Date: 2009-10-13 11:50
Message:
You will likely notice that a few unit tests are actually failing. This
means either the unit test for those bug reports contain bugs, or the
IFingerprinter implementation does, IMHO. This needs to be explored
separately.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=320024&aid=2877961&group_id=20024------------------------------------------------------------------------------
Come build with us! The BlackBerry(R) Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9 - 12, 2009. Register now!
http://p.sf.net/sfu/devconference_______________________________________________
Cdk-devel mailing list
Cdk-devel@...
https://lists.sourceforge.net/lists/listinfo/cdk-devel
