On 06/27/2012 05:24 PM, Markus Armbruster wrote: > Avi Kivity <a...@redhat.com> writes: > >> On 06/27/2012 04:28 PM, Eric Blake wrote: > [...] >>> You marked this function signature as returning uint64_t, but didn't >>> return anything. I think the logical return is the new contents of >>> *pvalue. Another logical choice would be marking the function void. >>> >> >> Void makes more sense here as you usually want in-place update. > > It's an assignment, so I'd follow the precedence of C's assignment > operator and return the new value of the left hand side.
Yup. -- error compiling committee.c: too many arguments to function
