On 10/28/2014 11:04 PM, Fam Zheng wrote:
> This returns the node name of a BDS. Remove the TODO comment and expect
> the callers to be explicit.
> 
> Signed-off-by: Fam Zheng <famz@redhat.com>
> ---
>  block.c               | 6 +++++-
>  include/block/block.h | 1 +
>  2 files changed, 6 insertions(+), 1 deletion(-)
> 

Can still return NULL because we don't have Jeff's patches for
auto-assigning a node name everywhere, right?  But that doesn't matter
in the context of this patch, so:

Reviewed-by: Eric Blake <eblake@redhat.com>

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org