Skip to content

Commit

Permalink
Update README.md
Browse files Browse the repository at this point in the history
  • Loading branch information
Ria9993 authored Aug 19, 2024
1 parent f37bc7b commit 528c584
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -66,11 +66,9 @@ Based on RFC 1459 : https://datatracker.ietf.org/doc/html/rfc1459
<br>

## + Review
다른 인원의 리뷰에서,
스마트 포인터는 오히려 해제 시점은 명확하게 하지 못하게 한다는 문제가 제기되었다.
해제시점이 명확하지만 스마트 포인터의 사용으로 가려지는 부분들이 몇 군데 있어, 이에 동의하였고
c/c++의 장점을 오히려 이용하지 못 한다는 느낌이라
지금 스마트 포인터로 삭제되는 채널은 로우한 포인터로 관리하는게 맞을 것 같다.
스마트 포인터가 오히려 해제 시점를 명확하게 하지 못하게 한다는 문제가 있다.
스마트 포인터의 사용으로 명시적 해제가 가려지는 부분들이 몇 군데 있어서
c/c++의 장점을 오히려 이용하지 못 한다는 느낌이라 채널같은 경우, 로우한 포인터로 관리하는게 맞을 것 같다.


# Architecture
Expand Down

0 comments on commit 528c584

Please sign in to comment.