Re: [PATCH 16/20] migration: Replace migration's JSON writer by the general one

2020-12-17 Thread Dr. David Alan Gilbert
* Markus Armbruster (arm...@redhat.com) wrote: > "Dr. David Alan Gilbert" writes: > > > * Markus Armbruster (arm...@redhat.com) wrote: > >> Commit 8118f0950f "migration: Append JSON description of migration > >> stream" needs a JSON writer. The existing qobject_to_json() wasn't a > >> good fit,

Re: [PATCH 16/20] migration: Replace migration's JSON writer by the general one

2020-12-17 Thread Dr. David Alan Gilbert
* Markus Armbruster (arm...@redhat.com) wrote: > "Dr. David Alan Gilbert" writes: > > > * Markus Armbruster (arm...@redhat.com) wrote: > >> Commit 8118f0950f "migration: Append JSON description of migration > >> stream" needs a JSON writer. The existing qobject_to_json() wasn't a > >> good fit,

Re: [PATCH 16/20] migration: Replace migration's JSON writer by the general one

2020-12-16 Thread Markus Armbruster
"Dr. David Alan Gilbert" writes: > * Markus Armbruster (arm...@redhat.com) wrote: >> Commit 8118f0950f "migration: Append JSON description of migration >> stream" needs a JSON writer. The existing qobject_to_json() wasn't a >> good fit, because it requires building a QObject to convert. Instead

Re: [PATCH 16/20] migration: Replace migration's JSON writer by the general one

2020-12-16 Thread Dr. David Alan Gilbert
* Markus Armbruster (arm...@redhat.com) wrote: > Commit 8118f0950f "migration: Append JSON description of migration > stream" needs a JSON writer. The existing qobject_to_json() wasn't a > good fit, because it requires building a QObject to convert. Instead, > migration got its very own JSON writ

[PATCH 16/20] migration: Replace migration's JSON writer by the general one

2020-12-11 Thread Markus Armbruster
Commit 8118f0950f "migration: Append JSON description of migration stream" needs a JSON writer. The existing qobject_to_json() wasn't a good fit, because it requires building a QObject to convert. Instead, migration got its very own JSON writer, in commit 190c882ce2 "QJSON: Add JSON writer". It