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

Boost.GIL 3 ideas from the past #55

Closed
chhenning opened this issue Mar 20, 2018 · 15 comments
Closed

Boost.GIL 3 ideas from the past #55

chhenning opened this issue Mar 20, 2018 · 15 comments
Assignees
Labels
cat/enhancement Improvements, but not fixes addressing identified bugs
Projects

Comments

@chhenning
Copy link
Contributor

chhenning commented Mar 20, 2018

These ideas are old and might be obsolete.

@chhenning chhenning added the cat/enhancement Improvements, but not fixes addressing identified bugs label Mar 20, 2018
@chhenning chhenning self-assigned this Mar 20, 2018
@chhenning chhenning changed the title gil 2.0 ideas from the past gil 3 ideas from the past Mar 20, 2018
@mloskot mloskot changed the title gil 3 ideas from the past Boost.GIL 3 ideas from the past Mar 20, 2018
@mloskot
Copy link
Member

mloskot commented Mar 20, 2018

I've taken the liberty to make the title and clearer, searchable and here too https://github.com/boostorg/gil/projects/2

Current master and develop are at Boost.GIL 2, correct?

@chhenning
Copy link
Contributor Author

Yes, I think so.

@mloskot
Copy link
Member

mloskot commented Mar 20, 2018

Right, thanks for clarification.

It seems a bit confusing to not to refer to Boost versions, unless there is <boost/gil/version.hpp, like in boost/beast/version.hpp or boost/spirit/version.hpp and others.

@chhenning
Copy link
Contributor Author

I have added version.hpp and raised the develop version to 2.2. Indicating the new extensions, like io and toolbox.

@mloskot
Copy link
Member

mloskot commented Mar 20, 2018

Sounds good!

@mloskot
Copy link
Member

mloskot commented Mar 20, 2018

make use of c++11 features

Although I would welcome complete switch to C++11 that opens possibility to refactor/simpliy the library implementation, I think this is too early to set requirement of C++11:

  • Does Boost allow to bump minimum requirement for an existing library? I'm not sure.
  • When that should happen, before or after the new IO is released? I'd vote for after.

/cc @stefanseefeld

@stefanseefeld
Copy link
Member

I'm not sure there are any fixed rules as to whether we are allowed to make C++11 a requirement. Let's discuss this based on specific goals: can you give a specific example where using a C++11 feature would help ?
I'm reluctant of adding dependencies on other boost libraries (in particular if they are not part of the core set of libraries, such as boost::fusion).

@stefanseefeld
Copy link
Member

yeah, definitely after The Grand Merge, which will happen after the current release is complete. I'm confident I'll have the documentation refactoring complete by then...

Any news on the checksum issue with variant=release ?

@mloskot
Copy link
Member

mloskot commented Mar 21, 2018

@stefanseefeld

I'm reluctant of adding dependencies on other boost libraries (in particular if they are not
part of the core set of libraries

I agree, unless optional.

Any news on the checksum issue with variant=release ?

Although cause of the problem is still a puzzle, I've made small progress and pin-pointed area of Boost.GIL where the issues happens. I will e-mail soon this week.

@stefanseefeld

Let's discuss this based on specific goals: can you give a specific example where using a C++11 feature would help ?

Get rid of boost::lambda, boost::bind (good idea in 2005, bad idea in 2015) for saner compiler errors, at least, I hope.

@chhenning
Copy link
Contributor Author

Moving this to a wiki.

@chhenning
Copy link
Contributor Author

@mloskot
Copy link
Member

mloskot commented Mar 22, 2018

Closing?

@chhenning
Copy link
Contributor Author

chhenning commented Mar 22, 2018

Issue is not the format I think. A wiki seems better.

@mloskot
Copy link
Member

mloskot commented Mar 22, 2018

Yes, so my question was if this shouldn't be closed, or you want to keep it open?

@mloskot
Copy link
Member

mloskot commented Mar 30, 2018

I think I broken the link by renaming the wiki page, here it is

https://github.com/boostorg/gil/wiki/Boost.GIL-3-Ideas

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cat/enhancement Improvements, but not fixes addressing identified bugs
Projects
Boost.GIL 3
  
Awaiting triage
Development

No branches or pull requests

3 participants