Opened 18 years ago
Last modified 18 years ago
#1012 closed defect (fixed)
libecwj2-3.3 bug with m_nNextLine?
Reported by: | warmerdam | Owned by: | warmerdam |
---|---|---|---|
Priority: | high | Milestone: | |
Component: | GDAL_Raster | Version: | unspecified |
Severity: | normal | Keywords: | |
Cc: |
Description
Attachments (1)
Change History (4)
comment:2 by , 18 years ago
I should note, I was using libecwj2 3.3 RC1, which I believe to be the most current public code.
comment:3 by , 18 years ago
I repeated the problem using the attached sample program, and was able to see that the m_nNextLine value was not reset between SetViews as it should have been, causing ReadLineBIL to always return NCSECW_READ_FAILED. The fix Frank suggested seemed logically correct. I have patched it as if(m_pECWFileView) { CNCSError Error; m_nNextLine = 0; ... in the code from line 942 (rather than using Frank's fix) since the JP2 case does not check to see if the SetView is successful before resetting the value. Simon and I have discussed the fix and can't foresee any regressions it might cause, as every SetView invalidates the old m_nNextLine value in any case.
Note:
See TracTickets
for help on using tickets.