>>> On 11.06.15 at 17:42, <paul.durr...@citrix.com> wrote:
> The struct just contains three methods and no data, so the name
> hvm_mmio_ops more accurately reflects its content. A subsequent patch
> introduces a new structure which more accurately warrants the name
> hvm_mmio_handler so doing the rename in this purely cosmetic patch avoids
> conflating functional and cosmetic changes in a single patch.
>
> Signed-off-by: Paul Durrant <paul.durr...@citrix.com>
Acked-by: Jan Beulich <jbeul...@suse.com>
albeit
> @@ -182,21 +182,21 @@ int hvm_mmio_intercept(ioreq_t *p)
>
> for ( i = 0; i < HVM_MMIO_HANDLER_NR; i++ )
> {
> - hvm_mmio_check_t check_handler =
> - hvm_mmio_handlers[i]->check_handler;
> + hvm_mmio_check_t check =
> + hvm_mmio_handlers[i]->check;
Does this still not fit on a single line?
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
