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.

Sign up for GitHub

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

Jump to bottom

new git pull 12/03/2021 @ 12:29 eST #833

Closed
maserowik opened this issue Dec 3, 2021 · 1 comment · Fixed by #839
Closed

new git pull 12/03/2021 @ 12:29 eST #833

maserowik opened this issue Dec 3, 2021 · 1 comment · Fixed by #839
Assignees
@EricClaeys

Comments

@maserowik
Copy link

I am not running a RPiHQ camera but during the install I see the following I have a ZWO camera but wanted to share what I see.

capture_RPiHQ.cpp: In function ‘int main(int, char**)’:
capture_RPiHQ.cpp:829:14: warning: variable ‘ImgExtraText’ set but not used [-Wu nused-but-set-variable]
char const *ImgExtraText = "";
^~~~~~~~~~~~
capture_RPiHQ.cpp:830:6: warning: variable ‘extraFileAge’ set but not used [-Wun used-but-set-variable]
int extraFileAge = 0; // 0 disables it
EricClaeys added a commit that referenced this issue Dec 4, 2021
@EricClaeys
capture_RPiHQ.cpp: compiler warning
a4d1ef1 
The "extra text" file hasn't yet been fully implemented in the RPiHQ capture program, but variables to activate it are there.  This causes compiler messages about "variable set but not used".  This is not an issue, but some users have mentioned it, thinking it's an error.
I added a dummy line to keep the compiler quiet.  This line will be removed when the "extra text" file is implemented.

Fixes #833.
@EricClaeys EricClaeys mentioned this issue Dec 4, 2021
Merged
@EricClaeys
Copy link
Collaborator

@maserowik, thanks Mike. The "extra text file" hasn't been fully implemented in the RPiHQ version. These message aren't an issue, but I just created a fix to eliminate the messages as I realize they can be concerning to people.

@EricClaeys EricClaeys self-assigned this Dec 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@EricClaeys EricClaeys

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

capture_RPiHQ.cpp: compiler warning AllskyTeam/allsky
2 participants
@maserowik @EricClaeys