> -----Original Message----- > From: Borislav Petkov [mailto:bp@xxxxxxxxx] > Sent: Sunday, November 01, 2015 8:59 PM > > On Sun, Nov 01, 2015 at 11:11:23AM +0000, Kweh, Hock Leong wrote: > > Hmm .... If I combine these 2 flags to become one as > > "NO_MORE_WRITE_ACTION" to better describing the situation, you Okay > > with it? > > I don't understand, why combine? > > Why not simply make UPLOAD_DONE a positive value: > > #define UPLOAD_DONE 1 > #define ERR_OCCURRED -1 > > 0 would obviously mean, no errors occurred whatsoever. > Hi Boris, This is not a return value to indicate what is going now. It is a flag used in "cap_info->index" which positive value has a meaning of index number. I am using the negative value for the flag which similar to the implementation of pointer & error pointer (ERR_PTR). Thanks & Regards, Wilson ��.n��������+%������w��{.n�����{����*jg��������ݢj����G�������j:+v���w�m������w�������h�����٥