Archived issue #0031622

Samples - update MFC Animation sample with proper frame updates

Open CASCADEOCCT:Samplesclosed7 public notes

Search issues

Description

MFC Animation sample in current state demonstrates broken animation logic - it ignores real elapsed time and blocks 3D viewer interactions.

Steps to reproduce

N/A

Public activity

7 archived notes

Participants are labeled by their role within this record.

01Commenter 3
Branch [archived branch] has been created by Author.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Sat Jun 20 23:00:17 2020 +0300

    0031622: Samples - update MFC Animation sample with proper frame updates
    
    Animation sample has been updated to:
    - use reuse AIS_ViewController for general viewer manipulations;
    - update animation using elapsed time;
    - do not block camera manipilations;
    - get rid of redundant controls.
02Author
03Commenter 4
Please, avoid adding additional levels in code:

void CAnimationView3D::update3dView()
{
  if (!myView.IsNull())
  {
    if (++myUpdateRequests == 1)
    {
      Invalidate (FALSE);
      UpdateWindow();
    }
  }
}
04Commenter 3
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
05Author
Remark has been applied.
06Commenter 1
Combination -
OCCT branch : [archived branch]
master SHA - [revision removed]
[revision removed]
Products branch : [archived branch] SHA - [revision removed]
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: 17164.36000000016 / 17160.400000000114 [+0.02%]
Products
Total CPU difference: 11173.720000000081 / 11197.400000000112 [-0.21%]
Windows-64-VC14:
OCCT
Total CPU difference: 18707.65625 / 18650.578125 [+0.31%]
Products
Total CPU difference: 13062.6875 / 13012.71875 [+0.38%]


Image differences :
No differences that require special attention

Memory differences :
No differences that require special attention
07Commenter 3
Branch [archived branch] has been deleted by Participant.

[revision removed]

Related records