Mailing-List: contact mahout-dev-help@lucene.apache.org; run by ezmlm
Precedence: bulk
Reply-To: mahout-dev@lucene.apache.org
Message-ID: <2100982120.1208567541723.JavaMail.jira@brutus>
Date: Fri, 18 Apr 2008 18:12:21 -0700 (PDT)
From: "Karl Wettin (JIRA)" <jira@apache.org>
To: mahout-dev@lucene.apache.org
Subject: [jira] Commented: (MAHOUT-25) Minor bugs/issues from code
 inspection
In-Reply-To: <180321913.1207364484288.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ https://issues.apache.org/jira/browse/MAHOUT-25?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12590660#action_12590660 ] 

Karl Wettin commented on MAHOUT-25:
-----------------------------------

bq. (Somebody holler if I am not following protocol)

You could just hit close and skip the resolve step. It's used in for instance in organisations where a tester reports a bug, coder fix and hit resolve, tester confirms it's all good and close the issue.

> 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.