mailRe: [patch #4130] Extended system test for providing list of spectra ID for NMRPipe seriesTab


Others Months | Index by Date | Thread Index
>>   [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Header


Content

Posted by Troels Emtekær Linnet on September 02, 2013 - 15:13:
Hi Edward.

So the question if, if it neccessary for the user to supply a 'int_col' list, when loading 
a NMRPipe SeriesTab file.

There is a check, which is made for the generic format list, where 'int_col' has been designed.
But it should not apply for NMRPipe SeriesTab. ?

The easiest fix for the user, is just to supply the list.
But the list is not used for anything yet, and could just create confusion.

Best
Troels



Troels Emtekær Linnet


2013/9/2 Edward d Auvergne <NO-REPLY.INVALID-ADDRESS@xxxxxxx>
Follow-up Comment #1, patch #4130 (project relax):

This patch does not change the test.  It passes before and after applying the
patch.  What is the purpose of this addition?  Why is it part of an existing
test and not a new test?  Are you trying to replicate bug #21076
(http://gna.org/bugs/?21076)?

Note that the patch is missing a newline at the end - there should be 2 empty
lines between functions/methods.

    _______________________________________________________

Reply to this item at:

  <http://gna.org/patch/?4130>

_______________________________________________
  Message sent via/by Gna!
  http://gna.org/



Related Messages


Powered by MHonArc, Updated Mon Sep 02 17:20:08 2013