Hey Rob,

I'm a bit unclear about what white space should be removed. Do you mean
just this segment, or for all the other tests? It's easier to represent the
array data for the matrix when it's broken up and aligned. What would be
the preferred formatting?

Thanks.


On Wed, Oct 9, 2013 at 1:31 PM, Rob Emanuele <notifications@github.com>wrote:

> In src/test/scala/geotrellis/raster/op/hydrology/FlowDirectionSpec.scala:
>
> > +      val r_extent = RasterExtent(Extent(0,0,1,1),1,1,ncols,nrows)
> > +      val e = IntArrayRasterData(Array[Int](8,5,6,
> > +                                            NODATA,5,6,
> > +                                            9,6,7),
> > +                            ncols,nrows)
> > +      val e_raster = Raster(e, r_extent)
> > +      val m = IntArrayRasterData(Array[Int](1,4,16,
> > +                                            NODATA,64,16,
> > +                                            1,64,32),
> > +                            ncols,nrows)
> > +      val m_raster = Raster(m, r_extent)
> > +      val e_computed = FlowDirection(e_raster)
> > +
> > +      assertEqual(e_computed, m_raster)
> > +    }
> > +
>
> Unneeded white space is a big pet peeve of mine, can you make sure there
> isn't any?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/geotrellis/geotrellis/pull/590/files#r6869627>
> .
>