[Liblas-commits] hg-main-tree: logic bug in determining if we can fit more points...

[liblas-commits at liblas.org]

details:   http://hg.libpc.orghg-main-tree/rev/55b2a8004225
changeset: 566:55b2a8004225
user:      Howard Butler <hobu.inc at gmail.com>
date:      Thu Apr 14 10:26:10 2011 -0500
description:
logic bug in determining if we can fit more points in the block.  This was causing lots of extra PointBuffer creation.

diffstat:

 src/drivers/oci/Iterator.cpp |  2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diffs (12 lines):

diff -r 69547d1d8d6c -r 55b2a8004225 src/drivers/oci/Iterator.cpp
--- a/src/drivers/oci/Iterator.cpp	Thu Apr 14 09:24:44 2011 -0500
+++ b/src/drivers/oci/Iterator.cpp	Thu Apr 14 10:26:10 2011 -0500
@@ -201,7 +201,7 @@
     while (bDidRead)
     {
         boost::uint32_t numReadThisBlock = m_block->num_points;
-        if ((numPointsRead + numReadThisBlock) > (data.getCapacity() - data.getNumPoints()))
+        if ((numReadThisBlock) > (data.getCapacity() - data.getNumPoints()))
         {
             // We're done.  We still have more data, but the 
             // user is going to have to request another buffer.