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

Make clean public APIs #160

Open
slaperche-scality opened this issue Feb 9, 2018 · 2 comments
Open

Make clean public APIs #160

slaperche-scality opened this issue Feb 9, 2018 · 2 comments
Assignees
@slaperche-scality
Labels
priority

Comments

@slaperche-scality
Copy link
Contributor

slaperche-scality commented Feb 9, 2018 •
edited
Loading

nttec.h is the "public" header that will be used by the users of the library, we should cleanup the included headers to be sure we only expose "public" API and not internals of the library
@vrancurel
Copy link
Contributor

nttec.h now

@slaperche-scality
Copy link
Contributor Author

Indeed, I'll update the issue

@slaperche-scality slaperche-scality changed the title cleanup ntl.h cleanup nttec.h Feb 10, 2018
@vrancurel vrancurel changed the title cleanup nttec.h Make clean public APIs Aug 2, 2018
slaperche-scality added a commit to slaperche-scality/quadiron that referenced this issue Aug 3, 2018
@slaperche-scality
cleanup quadiron.h
c87274a 
`quadiron.h` is the public header, as such it should only
contains/expose the public interface of the library (and not everything,
as that was the case).

Refs: scality#160
@slaperche-scality slaperche-scality mentioned this issue Aug 3, 2018
Merged
slaperche-scality added a commit to slaperche-scality/quadiron that referenced this issue Aug 13, 2018
@slaperche-scality
cleanup quadiron.h
b62df0b 
`quadiron.h` is the public header, as such it should only
contains/expose the public interface of the library (and not everything,
as that was the case).

Refs: scality#160
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@slaperche-scality slaperche-scality

Labels
priority
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@vrancurel @slaperche-scality