Skip to content

INTERNAL: Rename memory block macros#770

Closed
ing-eoking wants to merge 1 commit into
naver:developfrom
ing-eoking:macro
Closed

INTERNAL: Rename memory block macros#770
ing-eoking wants to merge 1 commit into
naver:developfrom
ing-eoking:macro

Conversation

@ing-eoking
Copy link
Copy Markdown
Collaborator

@ing-eoking ing-eoking commented Jun 14, 2024

🔗 Related Issue

⌨️ What I did

  • Memory Block 관련 매크로 이름을 간결하게 변경합니다.
    • 모든 Memory Block에 대한 공통적인 정보를 접근하는 경우는 MBLCK_[Name]
    • 한 Memory Block에 대하여 접근하는 경우는 [Name]_MBLCK
  • 현재 사용중이지 않은 매크로는 비활성화 시켰습니다.
- #define MBLCK_GET_NUMBLKS(l) ((l)->blck_cnt)
- #define MBLCK_GET_BODYLEN(l) ((l)->body_len)
  • 매크로 이름 변경 이유
    • 기존에 비해 가독성이 좋아집니다.
    • 이전 논의에서 현재 사용중인 매크로 이름과 사용방식 변경을 고려해보라는 의견이 있었습니다.

@ing-eoking ing-eoking requested a review from namsic June 14, 2024 03:00
@ing-eoking ing-eoking self-assigned this Jun 14, 2024
@jhpark816
Copy link
Copy Markdown
Collaborator

@ing-eoking
사용법 개선 없이 이름만 변경하는 것은 현재 의미가 없어 보입니다.

사용법 개선해야 할 사항이 있다면 이를 개선하시고,
사용법 개선에 따라 이름 변경이 필요할 경우에만 변경하면 됩니다.

@ing-eoking
Copy link
Copy Markdown
Collaborator Author

ing-eoking commented Jun 14, 2024

해당 PR은 의견 확인이 가장 중요했으며, 크게 의미없다는 의견이 나왔으므로 해당 PR은 종료하겠습니다.
만약 사용방식이 변경된다면 이에 대한 내용은 #756 에서 언급하도록 하겠습니다.

@ing-eoking ing-eoking closed this Jun 14, 2024
@ing-eoking ing-eoking removed the request for review from namsic June 14, 2024 08:34
@ing-eoking ing-eoking deleted the macro branch June 24, 2024 04:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants