search

Cavewhere / cavewhere

3D Cave Mapping Software
https://cavewhere.com
GNU General Public License v3.0
34 stars 8 forks source link

convert cwShot to allow for dive data #18

Open jlillest opened 9 years ago

jlillest commented 9 years ago

Dive data uses Distance, Compass/FromCompass, FromDepth/ToDepth

Make validator for this as well

jlillest commented 9 years ago

Ok, haven't fallen asleep on this. I've got a good part of this together and cleaned up but discovered last night that not only am I really bad at Qt Objects (what's wrong with the STLs?) but I also need to migrate a bunch of these changes out to the classes like cwSurveyChunky. Hope to make more progress tonight.

vpicaver commented 9 years ago

Sweet!

Most Qt containers support STD methods. For example: QList supports std::list style iterators, push_back(), empty(), etc… I like using Qt class instead of std stuff because when in roman speak roman. It's generally easier to integrate model class into GUI implementation went using Qt classes. Also runtimes and definitions for Qt containers and other class are well defined, where as the std are compiler dependent. Plus the Qt documentation is easier to read then std, in my opinion.

I look forward to the code you submit.

jlillest commented 9 years ago

This should probably be merged with issue #17, dunno if this is possible with github issues?