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

Separate private headers #62

Merged
merged 2 commits into from May 23, 2023
Merged

Separate private headers #62

merged 2 commits into from May 23, 2023

Conversation

pvuorela
Copy link
Contributor

Removed obsolete config-mkcal.h and avoiding installation of the more private headers. And avoid docs for private sqliteformat.h too.

@Tomin1 @dcaliste

@pvuorela pvuorela requested a review from Tomin1 May 23, 2023 10:51
Sqliteformat comments also adjusted to avoid any doxygen type of
document generation.
Sqlite format implementation.

This class implements the Sqlite format. It provides methods for
loading/saving/converting Sqlite format data into the internal
representation as Calendar and Incidences.
*/
// exported just for unit test. Would be better to avoid.
Copy link
Contributor

Choose a reason for hiding this comment

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

Actually, it can be dropped with #60 I think. I'll amend it in that direction.

Copy link
Contributor

@dcaliste dcaliste left a comment

Choose a reason for hiding this comment

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

Still more cleaning, that's good.

@pvuorela pvuorela merged commit 461ce46 into master May 23, 2023
@mlehtima mlehtima deleted the separate_private_headers branch April 22, 2024 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants