Ticket #127 (closed defect: fixed)

Opened 11 months ago

Last modified 11 months ago

alignment_single/alignment.c: pairalign_seq() assigns to bench_output[] array

Reported by: Nathan Weeks <weeks@…> Owned by:
Priority: minor Milestone: version 1.2
Component: alignment Version: version 1.1.1
Keywords: Cc:
Application Version: omp-tasks Blocked By:
Blocking: Sensitive: no
Needs experiment: no
Experimental observations: 
--- omp-tasks/alignment/alignment_single/alignment.c.orig       Fri Feb 11 10:50:37 2011
+++ omp-tasks/alignment/alignment_single/alignment.c    Sun Mar 20 17:18:11 2011@@ -522,7 +522,7 @@
       for (sj = si + 1; sj < nseqs; sj++) {
          m = seqlen_array[sj+1];
          if ( n == 0 || m == 0 ) {
-            bench_output[si*nseqs+sj] = (int) 1.0;
+            seq_output[si*nseqs+sj] = (int) 1.0;
          } else {
             int se1, se2, sb1, sb2, maxscore, seq1, seq2, g, gh;
             int displ[2*MAX_ALN_LENGTH+1];

Description

There seems to be an erroneous assignment to "bench_output" in function
alignment_single/alignment.c:pairalign_seq(); it looks like the assignment
should be to "seq_output", which is how it is implemented in
alignment_for/alignment.c.

Change History

Changed 11 months ago by Xavier Teruel <xavier.teruel@…>

  • status changed from new to closed
  • resolution set to fixed

(In [7923736b21dd453bc8ad93c4ba4484b271f61036]) Fixing pairalign_seq()'s wrong assingment in alignment omp-tasks single version; closes #127

Note: See TracTickets for help on using tickets.