[Liblas-commits] hg: remove SetDataRecordLength, which didn't exist in 1.2, so no...

[liblas-commits at liblas.org]

changeset 3dc0e60ff216 in /Volumes/Data/www/liblas.org/hg
details: http://hg.liblas.orghg?cmd=changeset;node=3dc0e60ff216
summary: remove SetDataRecordLength, which didn't exist in 1.2, so no consequence API-wise.  If you want to increase the data record length, you must add dimensions to a liblas::Schema

diffstat:

 include/liblas/lasheader.hpp |  13 +++++++------
 src/lasheader.cpp            |  31 +------------------------------
 2 files changed, 8 insertions(+), 36 deletions(-)

diffs (77 lines):

diff -r 0a9071b292e9 -r 3dc0e60ff216 include/liblas/lasheader.hpp
--- a/include/liblas/lasheader.hpp	Tue Aug 31 21:19:25 2010 -0500
+++ b/include/liblas/lasheader.hpp	Wed Sep 01 09:23:01 2010 -0500
@@ -233,13 +233,14 @@
     /// Set identifier of point data (record) format.
     void SetDataFormatId(PointFormatName v);
 
-    /// \todo To be documented
+    /// The length in bytes of each point.  All points in the file are 
+    /// considered to be fixed in size, and the PointFormatName is used 
+    /// to determine the fixed portion of the dimensions in the point.  Any 
+    /// other byte space in the point record beyond the liblas::Schema::GetBaseByteSize() 
+    /// can be used for other, optional, dimensions.  If no schema is 
+    /// available for the file in the form of a liblas.org VLR schema record,
+    /// These extra bytes are available via liblas::Point::GetExtraData().
     boost::uint16_t GetDataRecordLength() const;
-
-    // /// Set the length of the point format.  Evidently, there are 
-    // /// LAS files in the wild that contain point formats of sizes that 
-    // /// are different than the prescribed set specified in the specification.
-    // void SetDataRecordLength(boost::uint16_t v);
     
     /// Get total number of point records stored in the LAS file.
     boost::uint32_t GetPointRecordsCount() const;
diff -r 0a9071b292e9 -r 3dc0e60ff216 src/lasheader.cpp
--- a/src/lasheader.cpp	Tue Aug 31 21:19:25 2010 -0500
+++ b/src/lasheader.cpp	Wed Sep 01 09:23:01 2010 -0500
@@ -382,37 +382,12 @@
 liblas::PointFormatName Header::GetDataFormatId() const
 {
     return m_format.GetDataFormatId();
-    // if (ePointFormat0 == m_dataFormatId)
-    //     return ePointFormat0;
-    // else if (ePointFormat1 == m_dataFormatId)
-    //     return ePointFormat1;
-    // else if (ePointFormat2 == m_dataFormatId)
-    //     return ePointFormat2;
-    // else
-    //     return ePointFormat3;
+
 }
 
 void Header::SetDataFormatId(liblas::PointFormatName v)
 {
     m_format.SetDataFormatId(v);
-    
-    // m_dataFormatId = static_cast<uint8_t>(v);
-    
-    
-    // UpdateSchema();
-    // 
-    // if (ePointFormat0 == m_dataFormatId)
-    //     SetDataRecordLength(ePointSize0);
-    // else if (ePointFormat1 == m_dataFormatId) 
-    //     SetDataRecordLength(ePointSize1);
-    // else if (ePointFormat2 == m_dataFormatId)
-    //     SetDataRecordLength(ePointSize2);
-    // else if (ePointFormat3 == m_dataFormatId)
-    //     SetDataRecordLength(ePointSize3);
-    // else
-    //     SetDataRecordLength(ePointSize3);
-
-
 }
 
 uint16_t Header::GetDataRecordLength() const
@@ -420,10 +395,6 @@
     return m_format.GetByteSize();
 }
 
-// void Header::SetDataRecordLength( uint16_t v )
-// {
-//     // m_format.SetByteSize(v);
-// }
 uint32_t Header::GetPointRecordsCount() const
 {
     return m_pointRecordsCount;