On Sun, Mar 1, 2020 at 1:37 AM Bart Van Assche <bvanassche@xxxxxxx> wrote: > > On 2020-02-21 02:46, Jack Wang wrote: > > +enum rtrs_imm_type { > > + RTRS_IO_REQ_IMM = 0, /* client to server */ > > + RTRS_IO_RSP_IMM = 1, /* server to client */ > > + RTRS_IO_RSP_W_INV_IMM = 2, /* server to client */ > > + > > + RTRS_HB_MSG_IMM = 8, /* HB: HeartBeat */ > > + RTRS_HB_ACK_IMM = 9, > > + > > + RTRS_LAST_IMM, > > +}; > > This is the first time in this header file that the abbreviation "hb" is > used. Please add a comment that explains what this abbreviation stands for. > > Thanks, > > Bart. Will do, thank you