sol-prog/schm — pull request #5 by bo0ts (2011-12-07 00:09:07+00:00)
Title: PList modernized

Hi there,
I like your project, but the current state of the code is inappropriate, even for a hobby project.
The two commits refactor PList and annotate it with comments so you can use it as a base to explore the world of modern C++. They don't touch the algorithms although they seem very messy and inefficient. The first thing you should look at is const-correctness and references.
Watch out for include guards and you have a using directive in one of your headers polluting the global scope. After that have a look at the header of the stdlib and familiarize yourself with iterators.

There are some more advanced things in this commit: move semantics (a.k.a. &&) and defaulted constructors. You can safely ignore them for now but you should read something about constructor initializer lists to write proper destructors.

The largest flaw by far was the destructor. vector frees its resources on its destruction. No need for you to do it.

Have a look at stackoverflow.com for some great resources.
If you have any questions, don't hesitate to send me a message.

Cheers,
Philipp