zzh
/
gatk-3.8
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Another round of FindBugs fixes. Object internally stores a reference to an externally mutable array. Very dangerous.

This commit is contained in:
Ryan Poplin 2012-08-21 09:33:58 -04:00
parent 55b7949d68
commit 605acaae9c
3 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
protected/java/src/org/broadinstitute/sting/gatk/walkers/haplotypecaller/DeBruijnVertex.java
View File

@ -14,7 +14,7 @@ public class DeBruijnVertex {
public final int kmer;
public DeBruijnVertex( final byte[] sequence, final int kmer ) {
this.sequence = sequence;
this.sequence = sequence.clone();
this.kmer = kmer;
}

2
public/java/src/org/broadinstitute/sting/gatk/walkers/bqsr/StandardRecalibrationEngine.java
View File

@ -42,7 +42,7 @@ public class StandardRecalibrationEngine implements RecalibrationEngine, PublicP
protected RecalibrationTables recalibrationTables;
public void initialize(final Covariate[] covariates, final RecalibrationTables recalibrationTables) {
this.covariates = covariates;
this.covariates = covariates.clone();
this.recalibrationTables = recalibrationTables;
}

14
public/java/src/org/broadinstitute/sting/utils/Haplotype.java
View File

@ -55,22 +55,22 @@ public class Haplotype {
* @param bases bases
* @param qual qual
*/
public Haplotype(byte[] bases, int qual) {
this.bases = bases;
public Haplotype( final byte[] bases, final int qual ) {
this.bases = bases.clone();
quals = new double[bases.length];
Arrays.fill(quals, (double)qual);
}
public Haplotype(byte[] bases, double[] quals) {
this.bases = bases;
this.quals = quals;
public Haplotype( final byte[] bases, final double[] quals ) {
this.bases = bases.clone();
this.quals = quals.clone();
}
public Haplotype(byte[] bases) {
public Haplotype( final byte[] bases ) {
this(bases, 0);
}
public Haplotype(byte[] bases, GenomeLoc loc) {
public Haplotype( final byte[] bases, final GenomeLoc loc ) {
this(bases);
this.genomeLocation = loc;
}