HV: Do not return expression contains subexpression

yuchuyang · lijinxia · commit cb064b1133c0 · 2018-07-10T15:55:33.000+08:00
Operation should be done outside return expression.

Signed-off-by: Yang, Yu-chu &lt;yu-chu.yang@intel.com&gt;
diff --git a/hypervisor/debug/printf.c b/hypervisor/debug/printf.c
@@ -23,7 +23,8 @@ static int charout(int cmd, const char *s, int sz, void *hnd)
 		else
 			s += console_write(s, sz);
 
-		return (*nchars += (s - p));
+		*nchars += (s - p);
+		return *nchars;
 	}
 	/* fill mode */
 	else {

Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,8 @@ static int charout(int cmd, const char s, int sz, void hnd)`
`23`	`23`	`else`
`24`	`24`	`s += console_write(s, sz);`
`25`	`25`
`26`		`- return (*nchars += (s - p));`
	`26`	`+ *nchars += (s - p);`
	`27`	`+ return *nchars;`
`27`	`28`	`}`
`28`	`29`	`/* fill mode */`
`29`	`30`	`else {`