Skip to content
GitLab
Explore
Sign in
Primary navigation
Search or go to…
Project
R
RobotAPI
Manage
Activity
Members
Labels
Plan
Issues
Issue boards
Milestones
Code
Merge requests
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Build
Pipelines
Jobs
Pipeline schedules
Artifacts
Deploy
Container Registry
Model registry
Help
Help
Support
GitLab documentation
Compare GitLab plans
Community forum
Contribute to GitLab
Provide feedback
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
Software
ArmarX
RobotAPI
Merge requests
!139
Refactor inheritance in armem/core
Code
Review changes
Check out branch
Download
Patches
Plain diff
Merged
Refactor inheritance in armem/core
armem/refactor-inheritance
into
armem/dev
Overview
0
Commits
13
Pipelines
0
Changes
3
Merged
Rainer Kartmann
requested to merge
armem/refactor-inheritance
into
armem/dev
3 years ago
Overview
0
Commits
13
Pipelines
0
Changes
3
Expand
Remove common base classes and virtual inheritance
Move constructors and operators to base classes
Add tests for copy/move constructors and operators
0
0
Merge request reports
Viewing commit
0d934b0f
Prev
Next
Show latest version
3 files
+
57
−
35
Inline
Compare changes
Side-by-side
Inline
Show whitespace changes
Show one file at a time
Files
3
Search (e.g. *.vue) (Ctrl+P)
0d934b0f
Remove unnecessary overrides of copy(), add tests for checking whether copy() exists
· 0d934b0f
Rainer Kartmann
authored
3 years ago
source/RobotAPI/libraries/armem/core/base/EntityBase.h
+
0
−
27
Options
@@ -268,33 +268,6 @@ namespace armarx::armem::base
return
"entity"
;
}
virtual
_Derived
copy
()
const
override
{
_Derived
d
;
this
->
_copySelf
(
d
);
return
d
;
}
virtual
_Derived
copyEmpty
()
const
override
{
_Derived
d
;
this
->
_copySelfEmpty
(
d
);
return
d
;
}
protected
:
virtual
void
_copySelf
(
DerivedT
&
other
)
const
override
{
Base
::
_copySelf
(
other
);
}
virtual
void
_copySelfEmpty
(
DerivedT
&
other
)
const
override
{
Base
::
_copySelfEmpty
(
other
);
}
private
:
Loading