Skip to content
Snippets Groups Projects

rm DoubleDescriptor from code

Merged Mikhail Svechnikov requested to merge from_DoubleDescriptor_7 into main

The rest of DoubleDescriptor interfaces are replaced to DoubleProperty.

DoubleDescriptor is completely removed.

Steps of Descriptor refactoring:

  1. reduce usage of functional pointers set() and get() of DoubleDescriptor
  2. move Descriptor functionality to Property
  3. use Property instead of Descriptor
  4. remove Descriptor from code
  5. refactor serialization of Property
  6. replace Property macro definitions like DOUBLE_PROPERTY(intensity, Intensity) by explicit member with setter and getter

This MR does (1,3,4) for DoubleDescriptor

Merge request reports

Pipeline #81899 passed

Pipeline passed for 9f19e383 on from_DoubleDescriptor_7

Approved by

Merged by Mikhail SvechnikovMikhail Svechnikov 2 years ago (Dec 1, 2022 3:24pm UTC)

Merge details

  • Changes merged into main with 8e424714.
  • Deleted the source branch.

Pipeline #81904 passed

Pipeline passed for 8e424714 on main

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading