-
Notifications
You must be signed in to change notification settings - Fork 453
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix warnings and android issues #2245
Changes from 5 commits
027d6af
f694ba0
202fc96
5103b27
60e1d1c
d2e89b7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -811,6 +811,10 @@ class ModelMatch { | |
{ | ||
m_aircraftRegExp.optimize(); | ||
} | ||
|
||
virtual ~ModelMatch() | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. or "virtual ~ModelMatch = default" or better "~ModelMatch final = default" There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Right, default semantics is better :) There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Btw, there are lot of warnings like "warning: class with destructor marked 'final' cannot be inherited from [-Wfinal-dtor-non-final-class]". So marking destructor final causes more warnings. |
||
{ | ||
} | ||
|
||
virtual bool match(const QString &aircraft, const QString &manufacturer, QString &model) | ||
{ | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -24,7 +24,6 @@ | |
#include "aptdemodsink.h" | ||
|
||
APTDemodSink::APTDemodSink(APTDemod *packetDemod) : | ||
m_aptDemod(packetDemod), | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same remark as with channelpowersink.cpp There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ok |
||
m_channelSampleRate(APTDEMOD_AUDIO_SAMPLE_RATE), | ||
m_channelFrequencyOffset(0), | ||
m_magsqSum(0.0f), | ||
|
@@ -33,6 +32,8 @@ APTDemodSink::APTDemodSink(APTDemod *packetDemod) : | |
m_imageWorkerMessageQueue(nullptr), | ||
m_samples(nullptr) | ||
{ | ||
(void)packetDemod; | ||
|
||
m_magsq = 0.0; | ||
|
||
applySettings(m_settings, true); | ||
|
@@ -129,8 +130,6 @@ void APTDemodSink::feed(const SampleVector::const_iterator& begin, const SampleV | |
|
||
void APTDemodSink::processOneSample(Complex &ci) | ||
{ | ||
Complex ca; | ||
|
||
// FM demodulation | ||
double magsqRaw; | ||
Real deviation; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it should be removed from the constructor parameters
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok