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

Revert "constexpr version" #4496

Merged
merged 2 commits into from Jan 30, 2019
Merged

Conversation

vigsterkr
Copy link
Member

This reverts commit 46f7838.

due to constexpr after every commit the Version.h will change and thus
the whole ccache is contaminated as Version.h is included in SGObject.h

This reverts commit 46f7838.

due to constexpr after every commit the Version.h will change and thus
the whole ccache is contaminated as Version.h is included in SGObject.h
@vigsterkr vigsterkr requested a review from gf712 January 30, 2019 10:55
Copy link
Member

@gf712 gf712 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just need to add get_version_main in cpp

@@ -35,60 +34,35 @@ class Version
/** print version */
static void print_version();

/** get main version */
static constexpr const char* get_version_main() {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to keep the equivalent of this

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be fixed in the commit below...

Copy link
Member

@gf712 gf712 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@@ -54,6 +53,61 @@ void Version::print_version()
#endif
}

const char* Version::get_version_main()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cool thanks!

@vigsterkr vigsterkr merged commit b91af0c into shogun-toolbox:develop Jan 30, 2019
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.

None yet

2 participants