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

Fully remove boost dependency #44

Open
FabienDanieau opened this issue May 5, 2016 · 4 comments
Open

Fully remove boost dependency #44

FabienDanieau opened this issue May 5, 2016 · 4 comments

Comments

@FabienDanieau
Copy link

There is still a dependency to boost in falcon/gmtl/Misc/MatrixConvert.h
@FabienDanieau FabienDanieau changed the title Fully remove boost dependencies Fully remove boost dependency May 5, 2016
@qdot
Copy link
Member

qdot commented May 7, 2016

+1 :)

@FabienDanieau
Copy link
Author

It seems that only vectors and matrices are used from GMTL. I think we can find / do something lighter than having the whole library included.

@qdot
Copy link
Member

qdot commented May 7, 2016

Yeah, I honestly can't remember why I included it other than "it was there and did what I needed at the time".

@agirault agirault mentioned this issue May 10, 2016
Merged
@FabienDanieau
Copy link
Author

Well basically falcon/gmtl/Misc/MatrixConvert.h is not used anywhere so there is no really a dependency to boost. I guess we can leave it as it is

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@qdot @FabienDanieau