Hi
On Thu, Nov 29, 2018 at 8:11 PM Eduardo Habkost <ehabk...@redhat.com> wrote:
>
> On Tue, Nov 27, 2018 at 01:27:49PM +0400, Marc-André Lureau wrote:
> > Similarly to accel properties, move compat properties out of globals
> > registration, and apply the machine compat properties during
> > device_post_init().
> >
> > Signed-off-by: Marc-André Lureau <marcandre.lur...@redhat.com>
> > ---
> [...]
> > @@ -191,7 +190,7 @@ struct MachineClass {
> > const char *default_machine_opts;
> > const char *default_boot_order;
> > const char *default_display;
> > - GArray *compat_props;
> > + GPtrArray *compat_props;
>
> What are the advantages/disadvantages of GArray vs GPtrArray vs
> GList here? Why did you decide to change this from GArray to
> GPtrArray in v4?
GList would be less efficient than an array (extra pointer, memory
allocations/fragmentation/cache etc).
GPtrArray is similar to GArray, it is just specialized for storing void*.
--
Marc-André Lureau
