Colored prefix option only if colored output is enabled #449

meierj · 2021-07-24T07:21:30+02:00

meierj commented

2021-07-24 07:21:30 +02:00

added missing preprocessor directives to make colored output optional

* added missing preprocessor directives to make colored output optional

muellerr was assigned by meierj

2021-07-24 07:21:30 +02:00

meierj added 2 commits 2021-07-24 07:21:31 +02:00

5e960f118f renamed freertos includes

a918c672a5 fixed merge conflicts

muellerr requested changes 2021-07-27 10:03:25 +02:00

osal/FreeRTOS/MessageQueue.h Outdated

						
				@ -11,6 +11,8 @@

				#include "FreeRTOS.h"

				#include "queue.h"

				#include "FreeRTOS.h"

muellerr commented

2021-07-26 14:43:24 +02:00

double include

meierj marked this conversation as resolved

muellerr added 1 commit 2021-07-27 10:03:37 +02:00

3895806a2b Merge branch 'development' into meier/ColeredDebugOutput

muellerr changed title from ~~meier/ColeredDebugOutput~~ to Colored prefix option only if colored output is enabled

2021-07-27 10:06:12 +02:00

meierj added 2 commits 2021-07-27 10:10:38 +02:00

2489276350 removed double include

ebc9d99022 Merge branch 'meier/ColeredDebugOutput' of https://egit.irs.uni-stuttgart.de/fsfw/fsfw into meier/ColeredDebugOutput

muellerr requested changes 2021-07-27 10:13:49 +02:00

serviceinterface/ServiceInterfaceBuffer.cpp Outdated

						
				@ -175,2 +174,2 @@

				    this->colorPrefix = colorPrefix;

				}

				#if FSFW_COLORED_OUTPUT == 1

					void ServiceInterfaceBuffer::setAsciiColorPrefix(std::string colorPrefix) {

muellerr commented

2021-07-27 10:05:20 +02:00

Usually code in preprocessor defines has not been indented yet. I'd keep it consistent

meierj marked this conversation as resolved

unittest/tests/mocks/MessageQueueMockBase.h Outdated

						
				@ -4,3 +4,3 @@

				#include <fsfw/ipc/MessageQueueIF.h>

				#include <fsfw/ipc/MessageQueueMessage.h>

				#include <unittest/core/CatchDefinitions.h>

				#include <unittest/user/unittest/core/CatchDefinitions.h>

muellerr commented

2021-07-27 10:07:40 +02:00

I think this needs to remain #include <unittest/core/CatchDefinitions.h> but I'd need to check fsfw-test for that again.. Unless you are sure this is the correct include, I'd leave it the old way

I think this needs to remain `#include <unittest/core/CatchDefinitions.h>` but I'd need to check `fsfw-test` for that again.. Unless you are sure this is the correct include, I'd leave it the old way

meierj marked this conversation as resolved

meierj added 1 commit 2021-07-27 12:59:25 +02:00

3d80d5d036 added proposed changes

muellerr approved these changes 2021-07-27 13:07:03 +02:00

muellerr merged commit 2d667cfb95 into development

2021-07-27 13:07:16 +02:00

muellerr referenced this issue from a commit

2021-07-27 13:07:17 +02:00

Merge pull request 'Colored prefix option only if colored output is enabled' (#449) from meier/ColeredDebugOutput into development

muellerr deleted branch meier/ColeredDebugOutput

2021-07-27 13:07:23 +02:00

Sign in to join this conversation.