MantisBT
Mantis Bug Tracker Workflow

View Issue Details Jump to Notes ] Related Changesets ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0031226Open CASCADE[OCCT] OCCT:Foundation Classespublic2019-12-06 12:152020-12-26 14:52
Reportertizmaylo 
Assigned Tobugmaster 
PrioritynormalSeverityminor 
StatusverifiedResolutionfixed 
PlatformOSOS Version
Product Version[OCCT] 6.3.1 
Target Version[OCCT] 7.6.0*Fixed in Version 
Summary0031226: Foundation Classes - TCollection_AsciiString::IsRealValue() returns true if a string contains a non-real value
DescriptionThe method TCollection_AsciiString::IsRealValue() of the TCollection_AsciiString class returns true if a string contains a non-real value (e.g. "9999a9").
Steps To ReproduceNot required
TagsNo tags attached.
Test case numberNot required
Attached Files

- Relationships
related to 0030623closedbugmaster Draw Harness - support hex color codes within ViewerTest::ParseColor() 

-  Notes
(0089486)
abv (manager)
2019-12-13 07:10

This is historic implementation of method TCollection_AsciiString::IsRealValue() - it always returned true if the string was started with some characters that can be interpreted as integer or real value. I propose we can document this in the comment to the method, but it is better not to change behavior as it can break some existig code. Perhaps additional Boolean argument can be added indicating the need for a strict check.
(0097574)
git (administrator)
2020-12-14 15:18

Branch CR31226 has been created by mkrylova.

SHA-1: ae6b512bc2eb4cd7ec7b6fcd5273976e9a2572d9


Detailed log of new commits:

Author: mkrylova
Date: Mon Dec 14 15:18:43 2020 +0300

    0031226: Foundation Classes - TCollection_AsciiString::IsRealValue() returns true if a string contains a non-real value
    
    - Added comment to the method
    - Added Boolean argument indicating the need for a strict check
(0097647)
kgv (developer)
2020-12-16 17:51

+  //! theToCheckFull - argument indicating the need for a strict check is the AsciiString a real value.


Please use @param syntax for documenting arguments.
> argument indicating the need for a strict check is the AsciiString a real value.
when TRUE, checks if entire string defines a real value;
otherwise checks if string starts with a real value
(0097674)
git (administrator)
2020-12-17 13:19

Branch CR31226 has been updated by mkrylova.

SHA-1: 9fc09618f4b9f5debf4bc85e8775449a4ddfe8d7


Detailed log of new commits:

Author: mkrylova
Date: Thu Dec 17 13:21:18 2020 +0300

    kgv remarks:
    - changed description of TCollection_AsciiString::IsRealValue

(0097677)
git (administrator)
2020-12-17 13:21

Branch CR31226_1 has been created by mkrylova.

SHA-1: f604e77134f18414cdc58a20233ddf19c63bc35f


Detailed log of new commits:

Author: mkrylova
Date: Mon Dec 14 15:18:43 2020 +0300

    0031226: Foundation Classes - TCollection_AsciiString::IsRealValue() returns true if a string contains a non-real value
    
    - Added comment to the method
    - Added Boolean argument indicating the need for a strict check
(0097679)
git (administrator)
2020-12-17 14:12

Branch CR31226_1 has been updated by mkrylova.

SHA-1: 2307a213a06b461b8a1b95742e4e606977282cfe


Detailed log of new commits:

Author: mkrylova
Date: Thu Dec 17 14:14:18 2020 +0300

    # fixed code style

(0097680)
git (administrator)
2020-12-17 14:13

Branch CR31226_2 has been created by mkrylova.

SHA-1: 9af5fa9d337d43863964a57abc55b0804f3bd5fd


Detailed log of new commits:

Author: mkrylova
Date: Mon Dec 14 15:18:43 2020 +0300

    0031226: Foundation Classes - TCollection_AsciiString::IsRealValue() returns true if a string contains a non-real value
    
    - Added comment to the method
    - Added Boolean argument indicating the need for a strict check
(0097692)
kgv (developer)
2020-12-17 16:07
edited on: 2020-12-17 16:14

Please update existing places of IsRealValue() usage in ViewerTest to use new flag and add a test case.

(0097693)
kgv (developer)
2020-12-17 16:14

There is OCC11758 command in QABugs_19.cxx which can be extended to verify IsRealValue() in unit-test alike style.
(0097708)
git (administrator)
2020-12-18 14:24

Branch CR31226_2 has been updated by mkrylova.

SHA-1: e7095d340a4207acf79de9eafb6ad98a0b2ba4f2


Detailed log of new commits:

Author: mkrylova
Date: Fri Dec 18 14:26:35 2020 +0300

    # kgv remarks
     - Extended OCC11758 command in QABugs_19.cxx to verify IsRealValue()
     - Updated existing places of IsRealValue() usage in ViewerTest to use new flag

(0097709)
git (administrator)
2020-12-18 14:25

Branch CR31226_3 has been created by mkrylova.

SHA-1: b7305d518aed2100967240c67549d935e705749d


Detailed log of new commits:

Author: mkrylova
Date: Mon Dec 14 15:18:43 2020 +0300

    0031226: Foundation Classes - TCollection_AsciiString::IsRealValue() returns true if a string contains a non-real value
    
    - Added comment to the method
    - Added Boolean argument indicating the need for a strict
    - Extended OCC11758 command in QABugs_19.cxx to verify IsRealValue()
    - Updated existing places of IsRealValue() usage in ViewerTest to use new flag
(0097715)
kgv (developer)
2020-12-18 15:35

Please fix compiler warnings:

/dn62/builds/CR31226_3-master-KGV/OCCT_SRC/src/TCollection/TCollection_AsciiString.cxx: In member function 
'Standard_Boolean TCollection_AsciiString::IsRealValue(Standard_Boolean) const':
/dn62/builds/CR31226_3-master-KGV/OCCT_SRC/src/TCollection/TCollection_AsciiString.cxx:772:23: warning: 
NULL used in arithmetic [-Wpointer-arith]
[  2%] Building CXX object src/TKernel/CMakeFiles/TKernel.dir/__/Message/Message_Messenger.cxx.o
(0097723)
git (administrator)
2020-12-18 16:52

Branch CR31226_3 has been updated by mkrylova.

SHA-1: a639c52f2d5bd3017b97db7622c53b85a3abe113


Detailed log of new commits:

Author: mkrylova
Date: Fri Dec 18 16:53:58 2020 +0300

    # replaced NULL by '\0'

(0097724)
git (administrator)
2020-12-18 16:53

Branch CR31226_4 has been created by mkrylova.

SHA-1: 5f2599af2d18b1191c2d08bc8675f3ae8ddf3b73


Detailed log of new commits:

Author: mkrylova
Date: Mon Dec 14 15:18:43 2020 +0300

    0031226: Foundation Classes - TCollection_AsciiString::IsRealValue() returns true if a string contains a non-real value
    
    - Added comment to the method
    - Added Boolean argument indicating the need for a strict
    - Extended OCC11758 command in QABugs_19.cxx to verify IsRealValue()
    - Updated existing places of IsRealValue() usage in ViewerTest to use new flag
(0097733)
git (administrator)
2020-12-18 22:13

Branch CR31226_4 has been updated by kgv.

SHA-1: e8d9d28a97295977d5d0f8ffe8683b48b18c68de


Detailed log of new commits:

Author: kgv
Date: Fri Dec 18 22:14:28 2020 +0300

    # fix misprint in test case

(0097734)
kgv (developer)
2020-12-19 03:05

Please raise the patch
- OCCT branch: CR31226_4.

http://jenkins-test-12.nnov.opencascade.com:8080/view/CR31226_4-master-KGV/ [^]
(0097909)
bugmaster (administrator)
2020-12-26 12:14

Combination -
OCCT branch : WEEK-52
master SHA - 41046145c4a15f5cedf5f3c5877952ee00d568b4
a206de37fbfa0bf71bd534ae47192bbec23b8522
Products branch : WEEK-52 SHA - 290e5c74e8fef71947cadf90acb8e43c81ed10a1
was compiled on Linux, MacOS and Windows platforms and tested in optimize mode.

Number of compiler warnings:
No new/fixed warnings

Regressions/Differences/Improvements:
No regressions/differences

CPU differences:
Debian80-64:
OCCT
Total CPU difference: 17722.770000000193 / 17710.950000000055 [+0.07%]
Products
Total CPU difference: 12416.490000000109 / 12412.520000000126 [+0.03%]
Windows-64-VC14:
OCCT
Total CPU difference: 19321.5625 / 19274.6875 [+0.24%]
Products
Total CPU difference: 13844.421875 / 13853.046875 [-0.06%]


Image differences :
No differences that require special attention

Memory differences :
No differences that require special attention
(0097931)
git (administrator)
2020-12-26 14:52

Branch CR31226 has been deleted by inv.

SHA-1: 9fc09618f4b9f5debf4bc85e8775449a4ddfe8d7
(0097932)
git (administrator)
2020-12-26 14:52

Branch CR31226_1 has been deleted by inv.

SHA-1: 2307a213a06b461b8a1b95742e4e606977282cfe
(0097933)
git (administrator)
2020-12-26 14:52

Branch CR31226_2 has been deleted by inv.

SHA-1: e7095d340a4207acf79de9eafb6ad98a0b2ba4f2
(0097934)
git (administrator)
2020-12-26 14:52

Branch CR31226_3 has been deleted by inv.

SHA-1: a639c52f2d5bd3017b97db7622c53b85a3abe113
(0097935)
git (administrator)
2020-12-26 14:52

Branch CR31226_4 has been deleted by inv.

SHA-1: e8d9d28a97295977d5d0f8ffe8683b48b18c68de

- Related Changesets
occt: master d45edf24
Timestamp: 2020-12-14 12:18:43
Author: mkrylova
Committer: bugmaster
Details ] Diff ]
0031226: Foundation Classes - TCollection_AsciiString::IsRealValue() returns true if a string contains a non-real value

- Added comment to the method
- Added Boolean argument indicating the need for a strict
- Extended OCC11758 command in QABugs_19.cxx to verify IsRealValue()
- Updated existing places of IsRealValue() usage in ViewerTest to use new flag
mod - src/QABugs/QABugs_19.cxx Diff ] File ]
mod - src/TCollection/TCollection_AsciiString.cxx Diff ] File ]
mod - src/TCollection/TCollection_AsciiString.hxx Diff ] File ]
mod - src/ViewerTest/ViewerTest.cxx Diff ] File ]
mod - src/ViewerTest/ViewerTest_ObjectCommands.cxx Diff ] File ]
mod - src/ViewerTest/ViewerTest_RelationCommands.cxx Diff ] File ]
mod - src/ViewerTest/ViewerTest_ViewerCommands.cxx Diff ] File ]
mod - tests/bugs/mesh/bug26382 Diff ] File ]

- Issue History
Date Modified Username Field Change
2019-12-06 12:15 tizmaylo New Issue
2019-12-06 12:15 tizmaylo Assigned To => abv
2019-12-13 07:10 abv Note Added: 0089486
2020-09-11 16:13 utverdov Target Version 7.5.0 => 7.6.0*
2020-12-14 12:43 kgv Assigned To abv => mkrylova
2020-12-14 12:43 kgv Status new => assigned
2020-12-14 15:18 git Note Added: 0097574
2020-12-16 17:23 mkrylova Assigned To mkrylova => kgv
2020-12-16 17:23 mkrylova Status assigned => resolved
2020-12-16 17:23 mkrylova Steps to Reproduce Updated View Revisions
2020-12-16 17:51 kgv Note Added: 0097647
2020-12-16 17:51 kgv Assigned To kgv => mkrylova
2020-12-16 17:51 kgv Status resolved => assigned
2020-12-16 17:51 kgv Product Version => 6.3.1
2020-12-17 13:19 git Note Added: 0097674
2020-12-17 13:21 git Note Added: 0097677
2020-12-17 14:12 git Note Added: 0097679
2020-12-17 14:13 git Note Added: 0097680
2020-12-17 15:55 mkrylova Assigned To mkrylova => kgv
2020-12-17 15:55 mkrylova Status assigned => resolved
2020-12-17 16:04 kgv Relationship added related to 0030623
2020-12-17 16:07 kgv Note Added: 0097692
2020-12-17 16:07 kgv Assigned To kgv => mkrylova
2020-12-17 16:07 kgv Status resolved => assigned
2020-12-17 16:14 kgv Note Added: 0097693
2020-12-17 16:14 kgv Note Edited: 0097692 View Revisions
2020-12-18 14:24 git Note Added: 0097708
2020-12-18 14:25 git Note Added: 0097709
2020-12-18 14:25 mkrylova Assigned To mkrylova => kgv
2020-12-18 14:25 mkrylova Status assigned => resolved
2020-12-18 15:35 kgv Note Added: 0097715
2020-12-18 15:35 kgv Assigned To kgv => mkrylova
2020-12-18 15:35 kgv Status resolved => assigned
2020-12-18 16:52 git Note Added: 0097723
2020-12-18 16:53 git Note Added: 0097724
2020-12-18 16:54 mkrylova Assigned To mkrylova => kgv
2020-12-18 16:54 mkrylova Status assigned => resolved
2020-12-18 22:13 git Note Added: 0097733
2020-12-19 03:05 kgv Note Added: 0097734
2020-12-19 03:05 kgv Assigned To kgv => bugmaster
2020-12-19 03:05 kgv Status resolved => reviewed
2020-12-26 12:14 bugmaster Note Added: 0097909
2020-12-26 12:14 bugmaster Status reviewed => tested
2020-12-26 12:17 bugmaster Test case number => Not required
2020-12-26 14:40 bugmaster Changeset attached => occt master d45edf24
2020-12-26 14:40 bugmaster Status tested => verified
2020-12-26 14:40 bugmaster Resolution open => fixed
2020-12-26 14:52 git Note Added: 0097931
2020-12-26 14:52 git Note Added: 0097932
2020-12-26 14:52 git Note Added: 0097933
2020-12-26 14:52 git Note Added: 0097934
2020-12-26 14:52 git Note Added: 0097935


Copyright © 2000 - 2021 MantisBT Team
Powered by Mantis Bugtracker