View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0028367 | Open CASCADE | OCCT:DRAW | public | 2017-01-18 10:57 | 2017-09-29 16:24 |
| Reporter | kgv | Assigned To | apn | ||
| Priority | normal | Severity | minor | ||
| Status | closed | Resolution | fixed | ||
| Product Version | 7.1.0 | ||||
| Target Version | 7.2.0 | Fixed in Version | 7.2.0 | ||
| Summary | 0028367: Draw Harness - XSave should print an error on store failure | ||||
| Description | XSave command does not print any error on storage failure - TDocStd_Application::SaveAs() result is ignored. It does not look correct. | ||||
| Steps To Reproduce | N/A | ||||
| Tags | No tags attached. | ||||
| Test case number | Not needed | ||||
|
|
Branch CR28367 has been created by kgv. SHA-1: 72e723df75589f7980e09627295dd66792a627a2 Detailed log of new commits: Author: kgv Date: Wed Jan 18 10:59:35 2017 +0300 0028367: Draw Harness - XSave should print an error on store failure |
|
|
Patch is ready for review. |
|
|
OK, let it be like this right now. Duplicated methods in different OCAF draw-packages is another story. And receiving of more precised information about an error is also: there are too many ways to get advanced messages in different cases, so it must be collected in one method, I guess. Then it could be used. Reviewed, ready for testing. |
|
|
Branch CR28367 has been updated forcibly by apv. SHA-1: 1e5454695f1d22214423edfe15a83b53c47f69ab |
|
|
Branch CR28367 has been rebased on the current master |
|
|
Dear BugMaster, Branch CR28367 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested on Release mode. SHA-1: 1e5454695f1d22214423edfe15a83b53c47f69ab Number of compiler warnings: occt component : Linux: 0 (0 on master) Windows: 0 (0 on master) MacOS : 0 (0 on master) products component : Linux: 63 (63 on master) Windows: 0 (0 on master) MacOS : 1135 Regressions/Differences/Improvements: No regressions/differences Testing cases: Not needed Testing on Linux: occt component : Total MEMORY difference: 93331041 / 92055887 [+1.39%] Total CPU difference: 21386.320000000127 / 21062.120000000155 [+1.54%] products component : Total MEMORY difference: 31305592 / 31311912 [-0.02%] Total CPU difference: 5552.349999999974 / 5531.039999999989 [+0.39%] Testing on Windows: occt component : Total MEMORY difference: 58327453 / 58324216 [+0.01%] Total CPU difference: 19785.57562979855 / 19152.585972198493 [+3.30%] products component : Total MEMORY difference: 22558081 / 22520482 [+0.17%] Total CPU difference: 5505.868093799953 / 5374.920854399972 [+2.44%] There are no differences in images found by testdiff. |
|
|
Dear BugMaster, Branch CR28367 is TESTED. |
|
|
Branch CR28367 has been deleted by kgv. SHA-1: 1e5454695f1d22214423edfe15a83b53c47f69ab |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2017-01-18 10:57 | kgv | New Issue | |
| 2017-01-18 10:57 | kgv | Assigned To | => mpv |
| 2017-01-18 10:59 | git | Note Added: 0062735 | |
| 2017-01-18 10:59 | kgv | Note Added: 0062736 | |
| 2017-01-18 10:59 | kgv | Status | new => resolved |
| 2017-01-20 17:49 |
|
Note Added: 0062855 | |
| 2017-01-20 17:49 |
|
Assigned To | mpv => bugmaster |
| 2017-01-20 17:49 |
|
Status | resolved => reviewed |
| 2017-01-20 19:11 | git | Note Added: 0062874 | |
| 2017-01-20 19:12 |
|
Note Added: 0062876 | |
| 2017-01-20 19:14 |
|
Test case number | => Not needed |
| 2017-01-20 19:14 |
|
Assigned To | bugmaster => apv |
| 2017-01-23 12:54 |
|
Assigned To | apv => mkv |
| 2017-01-25 15:53 |
|
Note Added: 0063026 | |
| 2017-01-25 15:53 |
|
Note Added: 0063027 | |
| 2017-01-25 15:53 |
|
Assigned To | mkv => bugmaster |
| 2017-01-25 15:53 |
|
Status | reviewed => tested |
| 2017-01-27 16:15 | apn | Changeset attached | => occt master 61887a4a |
| 2017-01-27 16:15 | apn | Assigned To | bugmaster => apn |
| 2017-01-27 16:15 | apn | Status | tested => verified |
| 2017-01-27 16:15 | apn | Resolution | open => fixed |
| 2017-02-08 11:36 | git | Note Added: 0063616 | |
| 2017-09-29 16:21 |
|
Fixed in Version | => 7.2.0 |
| 2017-09-29 16:24 |
|
Status | verified => closed |
