[Kimchi-devel] [PATCH V9 1/7] bug fix: call a method should be followed by "()"

Royce Lv lvroyce at linux.vnet.ibm.com
Mon Apr 28 06:05:43 UTC 2014


Reviewed-by: Royce Lv<lvroyce at linux.vnet.ibm.com>
On 2014年04月25日 22:14, shaohef at linux.vnet.ibm.com wrote:
> From: ShaoHe Feng <shaohef at linux.vnet.ibm.com>
>
> it should be:
> FeatureTests.enable_screen_error_logging()
> instead of:
> FeatureTests.enable_screen_error_logging
>
> Signed-off-by: ShaoHe Feng <shaohef at linux.vnet.ibm.com>
> ---
>   src/kimchi/featuretests.py | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/kimchi/featuretests.py b/src/kimchi/featuretests.py
> index 07d5676..b1001ea 100644
> --- a/src/kimchi/featuretests.py
> +++ b/src/kimchi/featuretests.py
> @@ -171,7 +171,7 @@ class FeatureTests(object):
>                   # Libvirt requires adapter name, not needed when supports to FC
>                   return False
>           finally:
> -            FeatureTests.enable_screen_error_logging
> +            FeatureTests.enable_screen_error_logging()
>               pool is None or pool.undefine()
>               conn is None or conn.close()
>           return True




More information about the Kimchi-devel mailing list