MantisBT
Mantis Bug Tracker Workflow

View Issue Details Jump to Notes ] Related Changesets ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0028714Community[OCCT] OCCT:Application Frameworkpublic2017-05-04 06:322017-05-26 14:44
ReporterVico Liang 
Assigned Tompv 
PrioritynormalSeverityminor 
StatusfeedbackResolutionreopened 
PlatformOSOS Version
Product Version 
Target Version[OCCT] 7.2.0*Fixed in Version 
Summary0028714: Dimension of TDataStd_Real is not serialized to document
DescriptionNeigher bin nor xml document don't serialize the Dimension of TDataStd_Real.
Steps To ReproduceJust comments are added, not needed to be reproduced.
TagsNo tags attached.
Test case numberNot needed
Attached Files

- Relationships

-  Notes
(0066248)
git (administrator)
2017-05-18 11:28

Branch CR28714 has been created by mpv.

SHA-1: 161c549bef5171df1104910c52a5ee74a514582e


Detailed log of new commits:

Author: mpv
Date: Thu May 18 11:27:57 2017 +0300

    0028714: Dimension of TDataStd_Real is not serialized to document
    
    Added a comment to obsolete methods that these methods will be removed and that the corresponding field myDimension is not stored in the document.
(0066249)
mpv (developer)
2017-05-18 11:33

  Dear Vico,

These methods and field are obsolete and don't correspond to the main idea of OCAF component. All the attributes are atomic and must contain pure data, not specialized for some application.

So, I've added comments to methods that they will be deleted in some next version of OCCT.
(0066250)
kgv (developer)
2017-05-18 11:36
edited on: 2017-05-18 11:36

+  //! Obsolete method that will be removed in next versions.
+  //! This field is not supported in the persistence mechanism.
   Standard_EXPORT void SetDimension (const TDataStd_RealEnum DIM);

The methods which will be deleted in future are usually marked by Standard_DEPRECATED,
so that users will be able to actually see the usage of deprecated functionality in their code during compilation.

Or these methods beholds to internal API?

(0066252)
git (administrator)
2017-05-18 12:19

Branch CR28714 has been updated by mpv.

SHA-1: bceb3f1c3de0d7070756bc8dacdc68103f99cc5c


Detailed log of new commits:

Author: mpv
Date: Thu May 18 12:19:38 2017 +0300

    // added a Standard_DEPRECATED macro

(0066253)
mpv (developer)
2017-05-18 12:21

Thanks for remark, KGV. The Standard_DEPRECATED macro is added.
(0066345)
Vico Liang (developer)
2017-05-19 17:59

Dear mpv, thanks for your infomation. that's ok.
(0066485)
szy (administrator)
2017-05-23 16:55

Reviewed.
(0066486)
apv (tester)
2017-05-23 17:07

Dear BugMaster,

Please integrate branch CR28714 into the occt git-repository master.
(0066643)
bugmaster (administrator)
2017-05-26 14:44

MPV will think about possibility to avoid warnings during compilation of OCCT

http://jenkins-test-08.nnov.opencascade.com/view/IR-2017-05-25_IR-2017-05-25/view/OCCT%20compile/job/IR-2017-05-25_IR-2017-05-25-OCCT-Debian70-64-opt-compile/2/warnings21Result/ [^]

- Related Changesets
occt: master ea20d14b
Timestamp: 2017-05-18 08:27:57
Author: mpv
Committer: bugmaster
Details ] Diff ]
0028714: Dimension of TDataStd_Real is not serialized to document

Added a comment to obsolete methods that these methods will be removed and that the corresponding field myDimension is not stored in the document.

Added a Standard_DEPRECATED macro
mod - src/TDataStd/TDataStd_Real.hxx Diff ] File ]

- Issue History
Date Modified Username Field Change
2017-05-04 06:32 Vico Liang New Issue
2017-05-04 06:32 Vico Liang Assigned To => mpv
2017-05-16 15:35 mpv Assigned To mpv => Vico Liang
2017-05-16 15:35 mpv Assigned To Vico Liang => mpv
2017-05-16 15:35 mpv Status new => assigned
2017-05-18 11:28 git Note Added: 0066248
2017-05-18 11:33 mpv Note Added: 0066249
2017-05-18 11:33 mpv Assigned To mpv => szy
2017-05-18 11:33 mpv Status assigned => resolved
2017-05-18 11:33 mpv Steps to Reproduce Updated View Revisions
2017-05-18 11:36 kgv Note Added: 0066250
2017-05-18 11:36 kgv Note Edited: 0066250 View Revisions
2017-05-18 11:36 kgv Note Edited: 0066250 View Revisions
2017-05-18 12:19 git Note Added: 0066252
2017-05-18 12:21 mpv Note Added: 0066253
2017-05-19 17:59 Vico Liang Note Added: 0066345
2017-05-23 16:55 szy Note Added: 0066485
2017-05-23 16:55 szy Assigned To szy => bugmaster
2017-05-23 16:55 szy Status resolved => reviewed
2017-05-23 17:07 apv Note Added: 0066486
2017-05-23 17:07 apv Status reviewed => tested
2017-05-23 17:07 apv Test case number => Not needed
2017-05-26 14:28 bugmaster Changeset attached => occt master ea20d14b
2017-05-26 14:29 bugmaster Status tested => verified
2017-05-26 14:29 bugmaster Resolution open => fixed
2017-05-26 14:44 bugmaster Assigned To bugmaster => mpv
2017-05-26 14:44 bugmaster Note Added: 0066643
2017-05-26 14:44 bugmaster Status verified => feedback
2017-05-26 14:44 bugmaster Resolution fixed => reopened


Copyright © 2000 - 2017 MantisBT Team
Powered by Mantis Bugtracker