Client: fixup param type and return value

[gongchuang]

optimization param type of make_request;

Signed-off-by: gongchuang gong.chuang@zte.com.cn

[songbaisen]

@gongchuang First you should put your two commit into one commit.
second, here look like build error.

[yonghengdexin735]

@gongchuang hi, I think adding '-' to return value is unnecessary,
r = cfuse->init(newargc, newargv); if (r != 0) { cerr << "ceph-fuse[" << getpid() << "]: fuse failed to initialize" << std::endl; goto out_messenger_start_failed; }
here is the judgment for return value.

[songbaisen]

@yonghengdexin735 Wa,ou. Bang,Bang.

[jcsp]

Was there any particular reason for changing the sign of the error return codes? Was it breaking something?

The other (mds_rank) type change looks fine.

[gongchuang]

@jcsp no ohter reason,because i see FuseStore::start return negative value.for unified forms.

[jcsp]

@gongchuang those return codes get passed out as the status on the command line, which is a positive value: I don't think it's helpful to try and pass negative codes there.

Please remove the commit that changes the return values, and I'll merge this.

[jcsp]

@gongchuang ping

[gongchuang]

retest this please