Am 19.01.2011 15:17, schrieb Howard Butler:

On Jan 18, 2011, at 3:56 PM, Volker Wichmann wrote:

Hi,

I think there is a bug in libLAS 1.2.1 when retrieving point color
by random access (green and blue are interchanged), reader12.cpp,
line 334 ff:

detail::read_n(red, m_ifs, sizeof(uint16_t)); detail::read_n(blue,
m_ifs, sizeof(uint16_t)); detail::read_n(green, m_ifs,
sizeof(uint16_t));

I think this should read as

detail::read_n(red, m_ifs, sizeof(uint16_t)); detail::read_n(green,
m_ifs, sizeof(uint16_t)); detail::read_n(blue, m_ifs,
sizeof(uint16_t));

The same applies for lines 347 ff.



Volker,

I have confirmed this is fixed in libLAS 1.6.0b4 (we no longer read
fields this way either).

If you are interested, I could give you Mercurial push access if you
feel the need to continue to maintain a 1.2.x branch of libLAS.  I do
not have the resources to keep applying patches there, but I can
understand if others would like to keep it alive for the time being.

Howard


Howard,

although my time is limited too, I would agree to maintain the 1.2.x branch as long as SAGA GIS (and LIS) is using this version.

Volker
_______________________________________________
Liblas-devel mailing list
[email protected]
http://lists.osgeo.org/mailman/listinfo/liblas-devel

Reply via email to