diff options
author | Raquel Medina <raquel.medina@konsulko.com> | 2020-03-04 19:32:03 +0100 |
---|---|---|
committer | Raquel Medina <raquel.medina@konsulko.com> | 2020-03-09 15:03:20 +0100 |
commit | 5c750385d02116a92fa4c120ccc26abb8267bc97 (patch) | |
tree | a1ac931e48956a5d94e275447d061e630376d0aa /radio | |
parent | cd1a7f3bcbaef1e9c23549f472bce4ae42b2afad (diff) |
eliminate unneeded dependencies on public headers
- Remove unneeded dependencies from libqtappfw public
headers, specifically MessageEngine and Message
hierarchy header files, in preparation for refactoring
work on these areas.
This change helps shield client apps from the
upcoming changes in the library, and highlight the
separation between the library's core and the various
modules providing public interfaces to those apps.
- Make MessageType an scoped enum to fwd declare it.
- Update Copyright statement on files updated by
this patch.
Bug-AGL: SPEC-3112
Signed-off-by: Raquel Medina <raquel.medina@konsulko.com>
Change-Id: I12fd9cdc034aa1166f6448c07befadd9d2042b05
Diffstat (limited to 'radio')
-rw-r--r-- | radio/radio.cpp | 13 | ||||
-rw-r--r-- | radio/radio.h | 10 |
2 files changed, 14 insertions, 9 deletions
diff --git a/radio/radio.cpp b/radio/radio.cpp index 6813d08..1627cd7 100644 --- a/radio/radio.cpp +++ b/radio/radio.cpp @@ -1,5 +1,5 @@ /* - * Copyright (C) 2018 Konsulko Group + * Copyright (C) 2018-2020 Konsulko Group * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -14,11 +14,14 @@ * limitations under the License. */ +#include <QDebug> + #include "message.h" -#include "messageengine.h" -#include "radio.h" #include "radiomessage.h" #include "responsemessage.h" +#include "messageengine.h" +#include "radio.h" + Radio::Radio (QUrl &url, QQmlContext *context, QObject * parent) : QObject(parent), @@ -200,7 +203,7 @@ void Radio::onDisconnected() void Radio::onMessageReceived(MessageType type, Message *msg) { - if (msg->isEvent() && type == RadioEventMessage) { + if (msg->isEvent() && type == MessageType::RadioEventMessage) { RadioMessage *rmsg = qobject_cast<RadioMessage*>(msg); if (rmsg->isFrequencyEvent()) { @@ -224,7 +227,7 @@ void Radio::onMessageReceived(MessageType type, Message *msg) emit playingChanged(m_playing); } } - } else if (msg->isReply() && type == ResponseRequestMessage) { + } else if (msg->isReply() && type == MessageType::ResponseRequestMessage) { ResponseMessage *rmsg = qobject_cast<ResponseMessage*>(msg); if (rmsg->requestVerb() == "frequency_range") { diff --git a/radio/radio.h b/radio/radio.h index 642c58f..9e72a5a 100644 --- a/radio/radio.h +++ b/radio/radio.h @@ -1,5 +1,5 @@ /* - * Copyright (C) 2018 Konsulko Group + * Copyright (C) 2018-2020 Konsulko Group * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -17,11 +17,13 @@ #ifndef RADIO_H #define RADIO_H -#include <QDebug> #include <QObject> - #include <QtQml/QQmlContext> -#include "messageengine.h" + +class MessageEngine; +class Message; + +enum class MessageType; class Radio : public QObject { |