MantisBT - Community
View Issue Details
0025008Community[OCCT] OCCT:Visualizationpublic2014-06-15 13:392014-11-11 12:53
guidovanhilst 
bugmaster 
normaltrivial 
closedfixed 
win64win7
[OCCT] 6.7.1 
[OCCT] 6.8.0[OCCT] 6.8.0 
Not needed
0025008: Image_AlienPixMap - wrong PPM generated by ::savePPM() when image width is not equal to height
in Image_AlienPixMap::savePPM

in the second loop I think this should be SizeX() instead of SizeY() ?
(see code sniped below)

  for (Standard_Size aRow = 0; aRow < SizeY(); ++aRow)
  {
    for (Standard_Size aCol = 0; aCol < SizeY(); ++aCol)
No tags attached.
Issue History
2014-06-15 13:39guidovanhilstNew Issue
2014-06-15 13:39guidovanhilstAssigned To => san
2014-07-14 12:12sanNote Added: 0030112
2014-07-14 12:12sanStatusnew => resolved
2014-07-14 12:14sanNote Added: 0030113
2014-07-14 12:14sanAssigned Tosan => bugmaster
2014-07-14 12:14sanStatusresolved => reviewed
2014-07-14 12:14sanProduct Version => 6.7.1
2014-07-14 12:14sanTarget Version => 6.8.0
2014-07-14 13:05kgvSeveritymajor => trivial
2014-07-14 13:05kgvSummarywrong PPM generated in mage_AlienPixMap::savePPM => Image_AlienPixMap - wrong PPM generated by ::savePPM() when image width is not equal to height
2014-07-14 13:05kgvDescription Updatedbug_revision_view_page.php?rev_id=7712#r7712
2014-07-14 13:11kgvNote Added: 0030115
2014-07-14 14:58mkvAssigned Tobugmaster => mkv
2014-07-15 19:11mkvNote Added: 0030170
2014-07-15 19:11mkvTest case number => Not needed
2014-07-15 19:11mkvAssigned Tomkv => bugmaster
2014-07-15 19:11mkvStatusreviewed => tested
2014-07-22 15:13bugmasterChangeset attached => occt master a7b491fc
2014-07-22 15:13bugmasterStatustested => verified
2014-07-22 15:13bugmasterResolutionopen => fixed
2014-07-22 16:02gitNote Added: 0030318
2014-11-11 12:46aivFixed in Version => 6.8.0
2014-11-11 12:53aivStatusverified => closed

Notes
(0030112)
san   
2014-07-14 12:12   
Proposed correction applied in branch CR25008.
(0030113)
san   
2014-07-14 12:14   
Branch CR25008 reviewed, ready for testing.
(0030115)
kgv   
2014-07-14 13:11   
Dear bugmaster,

the patch does not affect OCCT built with FreeImage support.
Please check compilation and integrate.
(0030170)
mkv   
2014-07-15 19:11   
Dear BugMaster,

Branch CR25008 from occt git-repository (and master from products git-repository) was compiled on Linux and Windows platforms.
SHA-1: d6f021eb230ef707eb98e8a1337b68c553dec367

Number of compiler warnings:

occt component :
Linux: 15 (15 on master)
Windows: 0 (0 on master)

products component :
Linux: 11 (11 on master)
Windows: 2 (2 on master)
(0030318)
git   
2014-07-22 16:02   
Branch CR25008 has been deleted by inv.

SHA-1: d6f021eb230ef707eb98e8a1337b68c553dec367