[sheepdog] [sheepdog/sheepdog] c4e320: shared lib: drop unnecessary return value check
Vasiliy Tolstov
v.tolstov at selfip.ru
Tue Aug 11 08:48:28 CEST 2015
2015-08-11 9:19 GMT+03:00 Yu Yang <yuyanggg at gmail.com>:
> Hi, Hitoshi,
> I am sorry to say that those patches may not be merged directly. It may not work correctly.
>
> I check the patch here(c4e320b965fa47f74bbf0461e8c0043afef3d807) which is different from the original patch I submit in sheepdog-ng,
> I don't know why but I'm sure it cannot work as expect and import bugs. However , this is just one example and I didn't check others.
>
> And also as there is a bit different between sheepdog and sheepdog-ng, if those patches token from sheepdog-ng would be merged into sheepdog, some work need to be done and the code need be reviewed.
>
Sorry, but i don't understand how this
c250eee4e86df992f33ec625dfaa7dbf48c9d5b1 goes to sheepdog as
c4e320b965fa47f74bbf0461e8c0043afef3d807 can bring bugs. May be i miss
something, but sd_run_sdreq returns already req->ret, so returned
values means request status.
--
Vasiliy Tolstov,
e-mail: v.tolstov at selfip.ru
More information about the sheepdog
mailing list