CLEANUP: Refactor methods of setting ritems

[ing-eoking]

🔗 Related Issue

• jam2in/arcus-works#493

⌨️ What I did

• 추가 데이터를 담는 공간인 rlist를 추가했습니다.
• ritem_set_first / ritem_set_next, RTYPE 를 제거했습니다.
• 관련 함수 add_ritem / add_ritem_mblck / add_ritem_einfo / add_ritem_hinfo를 추가했습니다.
• conn_nread를 변경된 방식에 맞게 수정했습니다.
• rlist의 메모리 부족 처리를 추가하면서 일부 에러 핸들링을 아래의 PR처럼 통일시켰습니다.
  • #739

[jhpark816]

@namsic 이것도 리뷰가 완료되면, approve 해 주세요.

[ing-eoking]

@jhpark816 수정되었습니다.

[ing-eoking]

• add_ritem_e/hinfo
  • 해당 함수 또한 로직은 add_iov_e/hinfo_ascii/bin와 유사하다고 생각됩니다.
  • add_ritem_e/hinfo_ascii/bin으로 분리할까요?
  • add_ritem_mblck는 둘다(ASCII, Binary) 같은 방식을 사용하므로 분리 X

[namsic]

add_ritem_e/hinfo 또한 로직은 add_iov_e/hinfo_ascii/bin와 유사하다고 생각됩니다. add_ritem_e/hinfo_ascii/bin으로 분리할까요? add_ritem_mblck는 둘다(ASCII, Binary) 같은 방식을 사용하므로 분리 X

지금 구현 기준으로는 protocol 간 코드 차이가 거의 없어서 분리하지 않아도 될 것 같습니다.

[ing-eoking]

@jhpark816

수정되었습니다.

[ing-eoking]

@jhpark816 수정되었습니다.

[jhpark816]

@namsic 리뷰 바랍니다.

[jhpark816]

@namsic 리뷰를 마쳤나요? approved 상태여서 물어봅니다.

[namsic]

@jhpark816 리뷰를 마쳤나요? approved 상태여서 물어봅니다.

예. 리뷰 마쳤습니다. 개인적으로 아래 코드를 더 단순한 형태로 바꿀 수 있다면 좋을 것 같은데, 최선이다 싶은 방법이 떠오르지는 않습니다.

int i = 0;
if (einfo->nvalue > 0) {
} else {
    i++;
}
for (; i < einfo->naddnl && rltotal > 0; i++) {

[jhpark816]

개인적으로 아래 코드를 더 단순한 형태로 바꿀 수 있다면 좋을 것 같은데, 최선이다 싶은 방법이 떠오르지는 않습니다.

코드를 더 간단하고 이해하기 쉬운 형태로 변경해 주면 좋겠습니다.

[ing-eoking]

conn_nread를 1차적으로 변경했습니다.

[ing-eoking]

2차 수정되었습니다.