SharedPtr: add move operations

[kjbracey]

Summary of changes

Optimise SharedPtr by giving it move constructor and assignment operator.

Documentation

None

---

Pull request type

[X] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] Major update (Breaking change E.g. Return code change / API behaviour change)

---

Test results

[] No Tests required for this change (E.g docs only update)
[X] Covered by existing mbed-os tests (Greentea or Unittest)
[] Tests / results supplied as part of this PR

[ciarmcom]

@kjbracey-arm, thank you for your changes.
@ARMmbed/mbed-os-core @ARMmbed/mbed-os-maintainers please review.

[mbed-ci]

Test run: FAILED

Summary: 3 of 4 test jobs failed
Build number : 1
Build artifacts

Failed test jobs:

• jenkins-ci/mbed-os-ci_build-ARM
• jenkins-ci/mbed-os-ci_build-GCC_ARM
• jenkins-ci/mbed-os-ci_build-IAR

[0xc0170]

Errors in the build, [Error] main.cpp@68,15: [Pe309]: more than one instance of constructor "mbed::SharedPtr<T>::SharedPtr [with T=TestStruct]" matches the argument list:

[kjbracey]

Okay, I need to split this. I forgot there was the SharedPtr(nullptr_t) addition in here, which breaks people doing ptr = NULL or ptr = 0. That bit can follow the Callback changes - its NULL cleanup should cover it.

[adbridge]

@pan- @0xc0170 please review updates

[mbed-ci]

Test run: SUCCESS

Summary: 11 of 11 test jobs passed
Build number : 2
Build artifacts

[adbridge]

@kjbracey-arm is this all internal or technically a change in functionality ? Ie is this really a patch update ?

[kjbracey]

It's a refactor. Optimising existing copy operations to move when possible, rather than adding any new API.