-
Notifications
You must be signed in to change notification settings - Fork 12
Fix compile errors and warnings on some compilers #33
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
base: main
Are you sure you want to change the base?
Changes from all commits
2be856a
ac3e129
9e15c02
558e916
0851f70
3bd23c4
d45403e
2fe0e11
3fe4192
17dd841
6f6749c
92d991a
e7ed870
1fe6e82
459a50c
e65ea1b
a645df1
d212ebb
17879a0
9e0238f
098861c
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 |
|---|---|---|
| @@ -0,0 +1,9 @@ | ||
| Makefile.inc | ||
| dstar/Makefile.inc | ||
| **/*.o | ||
| **/*.a | ||
| starev | ||
| dstar/Normalisation | ||
| dstar/SeBa | ||
| **/*.data | ||
|
|
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -198,9 +198,11 @@ void put_state(double_state d, | |
| * make_profile -- | ||
| *----------------------------------------------------------------------------- | ||
| */ | ||
| /* Disabled because get_new_binary returns NULL, so this is broken | ||
|
Member
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. Does this need/warrant a fix?
Member
Author
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. No idea?
Member
Author
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. If neither of us know, is there anyone else who can answer that? |
||
| void make_profile(int id, real start_time, | ||
| double_profile& binary, double_init& init) { | ||
|
|
||
|
|
||
| cerr<<"double_support make_profile"<<endl; | ||
| real dt = (init.end_time - start_time)/init.n_steps; | ||
|
|
||
|
|
@@ -233,6 +235,7 @@ void make_profile(int id, real start_time, | |
| } | ||
|
|
||
| } | ||
| */ | ||
|
|
||
| void double_profile::init_double_profile(double_star* b, | ||
| star_state& p, star_state& s) { | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -119,7 +119,7 @@ double_state make_state(dyn* b) { | |
| #else | ||
|
|
||
|
|
||
| main() { | ||
| int main() { | ||
| cerr <<"Hello"<<endl; | ||
| } | ||
|
|
||
|
|
||
|
Member
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. was this a duplicate with a wrong name?
Member
Author
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. No, I think that the embedded starlab is only a partial copy, some (presumably unused) parts of it seem to have been removed, or were never copied in. It seems like the build system clean-up causes it to now try to build some things that weren't built before, and with parts of starlab missing, they're erroring out. Since SeBa doesn't use this anyway, I thought the easiest solution was just to remove it. |
This file was deleted.
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.
does this need to be fixed? or can we leave it out?
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.
Well, it requires something that's broken/disabled, so if it were used then we would have noticed that already, hopefully. SeBa simulates single stars right? So then a function for making a cluster is probably not needed.