Filip Roséen authored Mar 04, 2016 and Jean-Baptiste Kempf committed Mar 05, 2016

In order to provide better runtime safety manual management of dynamic
memory has been removed from `class Tags` and `class SimpleTags`.

This includes introducing usage of `std::vector` as well as
`std::string`, while also removing data-members that are no longer
required.

 - SimpleTag::b_default has been removed since it was written to (at
   once place), but never read; the write has been replaced by a comment.

 - The ParseSimpleTag function has been changed to return `bool` instead
   of a pointer to signal whether parsing was successful. The result
   will be written to `pout_simple` (instead of being indirectly
   returned through a pointer).

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>