ARMmbed
/
mbed-os
mirror of https://github.com/ARMmbed/mbed-os.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Construct control_t from a POD struct. Replace const control_t instances by this POD. 

This save memory and prevent inclusion of the constants in the binary if
they are not used.
pull/2714/head
Vincent Coubard 2016-08-26 14:01:58 +01:00 committed by Anna Bridge
parent a173382718
commit 99add3e08f
2 changed files with 44 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
features/frameworks/utest/source/utest_types.cpp
View File

@ -116,18 +116,20 @@ const char* utest::v1::stringify(utest::v1::status_t status)
} }
const utest::v1::control_t utest::v1::CaseNext(REPEAT_NONE, TIMEOUT_NONE); const utest::v1::base_control_t utest::v1::CaseNext = { REPEAT_NONE, TIMEOUT_NONE };
const utest::v1::control_t utest::v1::CaseNoRepeat(REPEAT_NONE); const utest::v1::base_control_t utest::v1::CaseNoRepeat = { REPEAT_NONE, TIMEOUT_UNDECLR };
const utest::v1::control_t utest::v1::CaseRepeatAll(REPEAT_ALL); const utest::v1::base_control_t utest::v1::CaseRepeatAll = { REPEAT_ALL, TIMEOUT_UNDECLR };
const utest::v1::control_t utest::v1::CaseRepeatHandler(REPEAT_HANDLER); const utest::v1::base_control_t utest::v1::CaseRepeatHandler = { REPEAT_HANDLER, TIMEOUT_UNDECLR };
const utest::v1::control_t utest::v1::CaseNoTimeout(TIMEOUT_NONE); const utest::v1::base_control_t utest::v1::CaseNoTimeout = { REPEAT_UNDECLR, TIMEOUT_NONE };
const utest::v1::control_t utest::v1::CaseAwait(TIMEOUT_FOREVER); const utest::v1::base_control_t utest::v1::CaseAwait = { REPEAT_UNDECLR, TIMEOUT_FOREVER };
const utest::v1::control_t utest::v1::CaseRepeat(CaseRepeatAll); // equal to CaeReapeatAll
const utest::v1::base_control_t utest::v1::CaseRepeat = { REPEAT_ALL, TIMEOUT_UNDECLR };
const utest::v1::control_t utest::v1::CaseRepeatHandlerOnly(CaseRepeatHandler); // equal to CaseRepeatHandler
const utest::v1::base_control_t utest::v1::CaseRepeatHandlerOnly = { REPEAT_HANDLER, TIMEOUT_UNDECLR };

49
features/frameworks/utest/utest/utest_types.h
View File

@ -120,6 +120,16 @@ namespace v1 {
/// Stringifies a status. /// Stringifies a status.
const char* stringify(utest::v1::status_t status); const char* stringify(utest::v1::status_t status);
/** POD version of the class control_t.
* It is used to instantiate const control_t objects as PODs
* and prevent them to be included in the final binary.
* @note: control_pod_t can be converted to control_t by copy construction.
*/
struct base_control_t {
repeat_t repeat;
uint32_t timeout;
};
/** Control class for specifying test case attributes /** Control class for specifying test case attributes
* *
* This class encapsulated control information about test cases which, when returned from * This class encapsulated control information about test cases which, when returned from
@ -148,18 +158,21 @@ namespace v1 {
* *
* In the future, more control information may be added transparently and backwards compatible. * In the future, more control information may be added transparently and backwards compatible.
*/ */
struct control_t struct control_t : private base_control_t
{ {
control_t() : repeat(REPEAT_UNDECLR), timeout(TIMEOUT_UNDECLR) {} control_t() : base_control_t(make_base_control_t(REPEAT_UNDECLR, TIMEOUT_UNDECLR)) {}
control_t(repeat_t repeat, uint32_t timeout_ms) : control_t(repeat_t repeat, uint32_t timeout_ms) :
repeat(repeat), timeout(timeout_ms) {} base_control_t(make_base_control_t(repeat, timeout_ms)) {}
control_t(repeat_t repeat) : control_t(repeat_t repeat) :
repeat(repeat), timeout(TIMEOUT_UNDECLR) {} base_control_t(make_base_control_t(repeat, TIMEOUT_UNDECLR)) {}
control_t(uint32_t timeout_ms) : control_t(uint32_t timeout_ms) :
repeat(REPEAT_UNDECLR), timeout(timeout_ms) {} base_control_t(make_base_control_t(REPEAT_UNDECLR, timeout_ms)) {}
control_t(base_control_t other) :
base_control_t(other) {}
control_t control_t
inline operator+(const control_t& rhs) const { inline operator+(const control_t& rhs) const {
@ -196,25 +209,31 @@ namespace v1 {
} }
private: private:
repeat_t repeat; static base_control_t make_base_control_t(repeat_t repeat, uint32_t timeout) {
uint32_t timeout; base_control_t result = {
repeat,
timeout
};
return result;
}
friend class Harness; friend class Harness;
}; };
/// does not repeat this test case and immediately moves on to the next one without timeout /// does not repeat this test case and immediately moves on to the next one without timeout
extern const control_t CaseNext; extern const base_control_t CaseNext;
/// does not repeat this test case, moves on to the next one /// does not repeat this test case, moves on to the next one
extern const control_t CaseNoRepeat; extern const base_control_t CaseNoRepeat;
/// repeats the test case handler with calling teardown and setup handlers /// repeats the test case handler with calling teardown and setup handlers
extern const control_t CaseRepeatAll; extern const base_control_t CaseRepeatAll;
/// repeats only the test case handler without calling teardown and setup handlers /// repeats only the test case handler without calling teardown and setup handlers
extern const control_t CaseRepeatHandler; extern const base_control_t CaseRepeatHandler;
/// No timeout, immediately moves on to the next case, but allows repeats /// No timeout, immediately moves on to the next case, but allows repeats
extern const control_t CaseNoTimeout; extern const base_control_t CaseNoTimeout;
/// Awaits until the callback is validated and never times out. Use with caution! /// Awaits until the callback is validated and never times out. Use with caution!
extern const control_t CaseAwait; extern const base_control_t CaseAwait;
/// Alias class for asynchronous timeout control in milliseconds /// Alias class for asynchronous timeout control in milliseconds
inline control_t CaseTimeout(uint32_t ms) { return ms; } inline control_t CaseTimeout(uint32_t ms) { return ms; }
@ -342,10 +361,10 @@ namespace v1 {
// deprecations // deprecations
__deprecated_message("Use CaseRepeatAll instead.") __deprecated_message("Use CaseRepeatAll instead.")
extern const control_t CaseRepeat; extern const base_control_t CaseRepeat;
__deprecated_message("Use CaseRepeatHandler instead.") __deprecated_message("Use CaseRepeatHandler instead.")
extern const control_t CaseRepeatHandlerOnly; extern const base_control_t CaseRepeatHandlerOnly;
__deprecated_message("Use REASON_NONE instead.") const failure_reason_t FAILURE_NONE = REASON_NONE; __deprecated_message("Use REASON_NONE instead.") const failure_reason_t FAILURE_NONE = REASON_NONE;
__deprecated_message("Use REASON_UNKNOWN instead.") const failure_reason_t FAILURE_UNKNOWN = REASON_UNKNOWN; __deprecated_message("Use REASON_UNKNOWN instead.") const failure_reason_t FAILURE_UNKNOWN = REASON_UNKNOWN;