You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mahout.apache.org by "Sean Owen (JIRA)" <ji...@apache.org> on 2008/04/19 02:56:21 UTC
[jira] Resolved: (MAHOUT-25) Minor bugs/issues from code inspection
[ https://issues.apache.org/jira/browse/MAHOUT-25?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Sean Owen resolved MAHOUT-25.
-----------------------------
Resolution: Fixed
(Somebody holler if I am not following protocol)
I committed my changes for this issue and am closing it.
> Minor bugs/issues from code inspection
> --------------------------------------
>
> Key: MAHOUT-25
> URL: https://issues.apache.org/jira/browse/MAHOUT-25
> Project: Mahout
> Issue Type: Bug
> Components: Matrix
> Affects Versions: 0.1
> Environment: All
> Reporter: Sean Owen
> Priority: Minor
> Fix For: 0.1
>
> Attachments: MAHOUT-25.patch
>
>
> Hi all, just started checking out the code base to get familiar with it and turned loose IntelliJ on the code. It picked up a few possible issues I thought I'd highlight:
> MatrixView:57
> for (int col = offset[COL]; col < offset[COL] + cardinality[COL]; row++)
> Pretty sure that should be col++ at the end.
> DenseMatrix:122
> Instances are compared uisng == instead of equals(). Doesn't matter if this is exactly the semantics you want, but if DenseMatrix ever defined a notion of equals() this wouldn't use it and might be a bug. Same in many other classes.
> Canopy:146, 168
> pointStronglyBound = pointStronglyBound | (dist < t2);
> Should this really be a non-shortcircuit, versus
> pointStronglyBound = pointStronglyBound || (dist < t2);
> or just
> if (!pointStronglyBound) {
> pointStronglyBound = dist < t2;
> }
> CanopyDriver:52,53
> String.valueOf(x) is a smidge faster than "" + x.
> Actually I've got several more but they're less important, like redundant casts or utility classes without private constructors, etc. I can keep going here... want to help polish a few things but don't want to get overbearing.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.