<table><tr><td style="">vkrause accepted this revision.<br />vkrause added a comment.<br />This revision is now accepted and ready to land.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D11974">View Revision</a></tr></table><br /><div><div><p>Simplifying the inheritance is perfectly fine, we do that in a number of other places too (e.g. we don't use "Thing"), let's just make this as complex as absolutely necessary.</p>

<p>Regarding tests, that's fine too, we don't need to test every single property access, as this is all the same code anyway. The purpose of datatypetest.cpp is primarily to ensure that the memory management of the polymorphic value types cannot be broken by "creative use" :)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R1003 KItinerary: Travel Reservation handling library</div></div></div><br /><div><strong>BRANCH</strong><div><div>add-more-datatypes (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11974">https://phabricator.kde.org/D11974</a></div></div><br /><div><strong>To: </strong>lbeltrame, vkrause<br /><strong>Cc: </strong>vkrause, KDE PIM, dvasin, winterz, mlaurent, knauss, dvratil<br /></div>