On 21 April 2014 15:39, Juan Quintela <quint...@redhat.com> wrote:
> After previous Peter patch, they are redundant.  This way we don't asign them
> except when needed.  Once there, there were lots of case where the ".fields"
> indentation was wrong:
>
>      .fields = (VMStateField []) {
> and
>      .fields =      (VMStateField []) {
>
> Change all the combinations to:
>
>      .fields = (VMStateField[]){
>
> The biggest problem (appart of aesthetics) was that checkpatch complained
> when we copy&pasted the code from one place to another.
>
> Signed-off-by: Juan Quintela <quint...@redhat.com>
> ---

Reviewed-by: Peter Maydell <peter.mayd...@linaro.org>

I'll hold this to apply to target-arm.next when you've got the dependent
patch into master.

thanks
-- PMM

Reply via email to