-
Notifications
You must be signed in to change notification settings - Fork 765
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 related to -Wshadow -Wnull-dereference -Wfloat-conversion -Wmissing-prototypes -Wmissing-declarations #495
Conversation
src/PJ_horner.c
Outdated
@@ -451,6 +451,10 @@ PJ *PROJECTION(horner) { | |||
complex_horner = 1; | |||
|
|||
Q = horner_alloc (degree, complex_horner); | |||
if( Q == 0 ) |
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.
Odd spacing.
src/PJ_healpix.c
Outdated
@@ -152,6 +152,7 @@ static int pnpoly(int nvert, double vert[][2], double testx, double testy) { | |||
* @param north_square the position of the north polar square (rHEALPix only) | |||
* @param south_square the position of the south polar square (rHEALPix only) | |||
**/ | |||
static |
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.
Why put these two on separate lines unlike the other ones added in this file?
src/PJ_robin.c
Outdated
{0.6213, -0.010418, 8.83923e-05, 6.24051e-06}, | ||
{0.5722, -0.00906601, 0.000182, 6.24051e-06}, | ||
{0.5322, -0.00677797, 0.000275608, 6.24051e-06} | ||
{1, 2.2199e-17f, -7.15515e-05f, 3.1103e-06f}, |
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.
Didn't change this 1
like the other below.
src/p_series.c
Outdated
|
||
/* FIXME: put the declaration in a header. Also used in gen_cheb.c */ | ||
void p_series(Tseries *T, FILE *file, char *fmt); | ||
|
||
void |
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.
Maybe fix this extra indent like the other one, too?
src/rtodms.c
Outdated
r = floor(r / 60.); | ||
deg = r; | ||
deg = (int)r; |
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.
What's up with the odd spacing here? Mixing tab and space?
…nr -Wmissing-prototypes -Wmissing-declarations
3f25c4b
to
002f444
Compare
002f444
to
a2303c2
Compare
…plicit conversions
a2303c2
to
8eb46db
Compare
93b6b06
to
abb21ad
Compare
c9f12e0. Issue raised by VS12
c7ea684
to
b885fb6
Compare
b885fb6
to
0cbb3c2
Compare
@@ -59,9 +60,13 @@ void* thread_main(void* unused) | |||
int main(int argc, char* argv[]) |
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.
Could just be int main(void)
.
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.
Good point. Fixed in master now
No description provided.