Skip to content
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

Allow plugins to be compiled as static libs, part 2. #2254

Merged
merged 1 commit into from
Sep 20, 2024

Conversation

srcejon
Copy link
Collaborator

@srcejon srcejon commented Sep 17, 2024

These are the plugins that have embedded .qrc data. The files needed to be renamed so they are unique.

ADS-B: Allow plugin to be compiled as static lib. Allow building without texttospeech, location or process.
VOR localizer: Allow plugin to be compiled as static lib.
SID feature: Allow plugin to be compiled as static lib.
Satellite Tracker: Allow building without texttospeech or timezone.
Map: Allow plugin to be compiled as static lib. Allow building without SSL.
APT: Allow plugin to be compiled as static lib.
Radio Astronomy: Allow plugin to be compiled as static lib. Allow building without process.
ChannelWebAPIUtils: Fix memory leaks.
DeviceSampleSource: Add signals that indicate when the position or direction of a device (or its antenna) has changed. This is to support per device position, which is useful for remote devices not at My Position (E.g. Kiwi's / SpyServers etc). More to come in a later patch.

…out texttospeech, location or process.

VOR localizer: Allow plugin to be compiled as static lib.
SID feature: Allow plugin to be compiled as static lib.
Satellite Tracker: Allow building without texttospeech or timezone.
Map: Allow plugin to be compiled as static lib. Allow building without SSL.
APT: Allow plugin to be compiled as static lib.
Radio Astronomy: Allow plugin to be compiled as static lib. Allow building without process.
ChannelWebAPIUtils: Fix memory leaks.
DeviceSampleSource: Add signals that indicate when the position or direction of a device (or it's antenna) has changed. This is to support per device position, which is useful for remote devices not at My Position.
Copy link

sonarcloud bot commented Sep 17, 2024

@srcejon
Copy link
Collaborator Author

srcejon commented Sep 17, 2024

Can we modify the SonarCloud rules?

E.g. for:

Use the init-statement to declare "deviceReport" inside the if statement.

we aren't using c++17 and the suggested fix is certainly not more readable, as claimed, IMO.

@f4exb
Copy link
Owner

f4exb commented Sep 20, 2024

I was thinking of upgrading to c++17 as most compilers now are at least at this level

@f4exb f4exb merged commit 7d1beb9 into f4exb:master Sep 20, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants