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

What is "QPoint::double" in doc/example_adapting_a_legacy_geometry_object_model.qbk ? #376

Closed
cirosantilli opened this Issue Jan 9, 2017 · 3 comments

Comments

Projects
None yet
4 participants
@cirosantilli

cirosantilli commented Jan 9, 2017

{ typedef QPoint::double type; };

In particular, fails to compile with the definition:

Removing the QPoint:: part works.

Sorry if a C++ / library feature that Google could not explain to me.

@awulkiew

This comment has been minimized.

Member

awulkiew commented Jan 9, 2017

Yes, it's probably a mistake in the example. I guess that original author at the beginning defined some coordinate type typedef in QPoint and at some point automatically changed the name to double. I'll fix it, thanks!

@awulkiew awulkiew closed this Jan 9, 2017

@mloskot

This comment has been minimized.

Member

mloskot commented Jan 9, 2017

For the record, I guess here is the original QPoint adoption:
https://gist.github.com/anonymous/1798839 (with variant here https://gist.github.com/meastp/1852693)
developed by @meastp as part of this thread
http://boost-geometry.203548.n3.nabble.com/Using-Boost-Geometry-with-a-legacy-class-hierarchy-td3734832.html

That initial version includes

template<> struct coordinate_type<QPoint>
{ typedef double type; };
@meastp

This comment has been minimized.

Contributor

meastp commented Jan 10, 2017

@cirosantilli, @awulkiew : It is a typo, and should be just double instead of QPoint::double. :)

@awulkiew awulkiew added the question label Jan 10, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment