Archived issue #0026637
Save function works incorrect in OCAF mfc sample
Description
There is exception trying to save document in .std format.
Steps to reproduce
Run mfc Ocaf sample, build shape, File-Save-.std format
Public activity
12 archived notes
Participants are labeled by their role within this record.
Branch [archived branch] has been created by Participant.
[revision removed]
Detailed log of new commits:
Author: ski
Date: Thu Sep 3 15:29:01 2015 +0300
0026637: Save function works incorrect in OCAF mfc sample
Storage format was corrected.
[revision removed]
Detailed log of new commits:
Author: ski
Date: Thu Sep 3 15:29:01 2015 +0300
0026637: Save function works incorrect in OCAF mfc sample
Storage format was corrected.
Dear Andrey,
Storage format was corrected in "Save" function in OCAF mfc sample.
Crash does not appear.
Please, review.
Storage format was corrected in "Save" function in OCAF mfc sample.
Crash does not appear.
Please, review.
Branch [archived branch] has been updated forcibly by Participant.
[revision removed]
[revision removed]
The fix is actual only for occt 6.9.1.
Note that it is not a regression, occt 6.9.0 demonstrates the same behavior.
Branch CR26637 was rebased on current state of occt 6.9.1.
Note that it is not a regression, occt 6.9.0 demonstrates the same behavior.
Branch CR26637 was rebased on current state of occt 6.9.1.
Branch [archived branch] has been created by Participant.
[revision removed]
Detailed log of new commits:
Author: ski
Date: Tue Sep 8 12:49:19 2015 +0300
0026637: Save function works incorrect in OCAF mfc sample
Format ".std" was eliminated from OCAF mfc sample.
[revision removed]
Detailed log of new commits:
Author: ski
Date: Tue Sep 8 12:49:19 2015 +0300
0026637: Save function works incorrect in OCAF mfc sample
Format ".std" was eliminated from OCAF mfc sample.
Branch CR26637_1 was created.
It contains eliminating of "std" format from OCAF mfc sample.
It should be integrated only into occt 7.0.
It contains eliminating of "std" format from OCAF mfc sample.
It should be integrated only into occt 7.0.
Reviewed, please integrate
Dear Commenter 1,
Branch CR26637_1 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested on Release mode.
[revision removed]
Number of compiler warnings:
occt component :
Linux: 15 (15 on master)
Windows: 0 (0 on master)
products component :
Linux: 39 (39 on master)
Windows: 0 (0 on master)
Regressions/Differences/Improvements:
No regressions/differences
Testing cases:
Not needed
Testing on Linux:
occt component :
Total MEMORY difference: 91836369 / 91937509 [-0.11%]
Total CPU difference: 17646.29999999888 / 17713.54999999897 [-0.38%]
products component :
Total MEMORY difference: 25229131 / 25220029 [+0.04%]
Total CPU difference: 7012.210000000008 / 6936.800000000004 [+1.09%]
Testing on Windows:
occt component :
Total MEMORY difference: 57156801 / 57146693 [+0.02%]
Total CPU difference: 16582.859499699112 / 16584.80951219933 [-0.01%]
products component :
Total MEMORY difference: 16226690 / 16225943 [+0.00%]
Total CPU difference: 5413.390700999969 / 5398.88260799997 [+0.27%]
There are no differences in images found by testdiff.
Branch CR26637_1 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested on Release mode.
[revision removed]
Number of compiler warnings:
occt component :
Linux: 15 (15 on master)
Windows: 0 (0 on master)
products component :
Linux: 39 (39 on master)
Windows: 0 (0 on master)
Regressions/Differences/Improvements:
No regressions/differences
Testing cases:
Not needed
Testing on Linux:
occt component :
Total MEMORY difference: 91836369 / 91937509 [-0.11%]
Total CPU difference: 17646.29999999888 / 17713.54999999897 [-0.38%]
products component :
Total MEMORY difference: 25229131 / 25220029 [+0.04%]
Total CPU difference: 7012.210000000008 / 6936.800000000004 [+1.09%]
Testing on Windows:
occt component :
Total MEMORY difference: 57156801 / 57146693 [+0.02%]
Total CPU difference: 16582.859499699112 / 16584.80951219933 [-0.01%]
products component :
Total MEMORY difference: 16226690 / 16225943 [+0.00%]
Total CPU difference: 5413.390700999969 / 5398.88260799997 [+0.27%]
There are no differences in images found by testdiff.
Dear Commenter 1,
mfc Ocaf sample was tested, it work correct.
mfc Ocaf sample was tested, it work correct.
Dear Commenter 1,
Branch CR26637_1 is TESTED.
Branch CR26637_1 is TESTED.
Branch [archived branch] has been deleted by Participant.
[revision removed]
[revision removed]
Branch [archived branch] has been deleted by Participant.
[revision removed]
[revision removed]