On Thu, Aug 20, 2015 at 09:58:11PM +0200, Michael Schaller wrote:
> Friendly ping to Julian.  I hope your exams went well...
> 

Hi Michael, I think it looks OK, but let me suggest some
changes:

- Drop the parent_version property and use _version directly
- Drop the cache property and use _cache directly
- Drop the TargetVersionList. AFAIK, the only thing different
  is it wrapping the list representation in a <TargetVersionList>
  thing, this does not seem to be needed (there's no point
  creating our own list type just for some corner-case
  cosmetic issue).

Then the patch will be much smaller, and still be able to do
the same.
-- 
Julian Andres Klode  - Debian Developer, Ubuntu Member

See http://wiki.debian.org/JulianAndresKlode and http://jak-linux.org/.

Be friendly, do not top-post, and follow RFC 1855 "Netiquette".
    - If you don't I might ignore you.

Reply via email to