removed spurious subsetting in spectrapply - see issue #109

lgatto · Jul 1, 2016 · c974b3b · c974b3b
1 parent 617c477
commit c974b3b
Showing 1 changed file with 0 additions and 5 deletions.
diff --git a/R/methods-OnDiskMSnExp.R b/R/methods-OnDiskMSnExp.R
@@ -312,11 +312,6 @@ setMethod("spectrapply", "OnDiskMSnExp",
     ## Check if we would do better with serial processing:
     BPPARAM <- getBpParam(object, BPPARAM = BPPARAM)
 
-    ## @lgatto: I suppose we will do the subsetting all on the object itself, thus
-    ## we don't need the subsetting code below, right?
-    ## Sub-set the feature data based on the arguments.
-    fd <- .subsetFeatureDataBy(fData(object), index=index, scanIdx=scanIdx,
-                               name=name, rtlim=rtlim)
     isOK <- .validateFeatureDataForOnDiskMSnExp(fd)
     if(!is.null(isOK))
         stop(isOK)