Stlink: 통합 변수 유형으로 코드 정리

에 만든 2020년 04월 06일 · 6코멘트 · 출처: stlink-org/stlink

내가 볼 수 있는 한 많은 함수가 고정 길이 변수를 사용합니다. 고정 데이터 형식을 처리해야 하는 함수에 실제로 유용합니다. 그러나 고정 길이 변수를 남용하면 잠재적인 빌드 실패와 불필요한 캐스트가 발생합니다.

예를 들어 src/usb.c에서:

send_recv는 txsize 및 rxsize 를 size_t 하지만 실제 컨텍스트에서 이러한 인수는 int 또는 unsigned int 없이 캐스팅되어야 합니다. 고정 길이, 호출하는 사람은 다른 유형의 인수를 제공합니다(이 경우 uint32_t 또는 매직 넘버). 32비트에서 64비트로의 의도하지 않은 캐스트는 호출 프로세스에서 적용됩니다. 또한 빌드 실패를 방지하기 위해 많은 명시적 캐스트가 필요합니다.

실제 사용법에 따라 고정 길이 변수를 교체하는 것이 좋습니다.
휴대성을 높이기 위해 퍼지 상태를 유지합니다.

다른 분들도 의미가 있다면 해보고 싶어요.

codfeedback codrefactoring needissuer-feedback

출처

chenguokai

👍1

모든 6 댓글

이것을 의제에 올려주셔서 감사합니다. 코드베이스 전체에 걸친 일반적인 문제이므로 v1.6.1에 바로 적용해야 하는지 현재로서는 확실하지 않습니다. @martonmiklos는 이전에 v1.6.2 마일스톤에 푸시한 코드 스타일과 관련된 일부 정리를 요청했습니다. 아마도 이것은 이 문제에 대한 좋은 장소가 될 수도 있지만 조금 더 나아가게 될 것입니다.

Nightwalker-87 에 2020년 04월 06일

먼저 로컬에서 일부 정리를 수행하고 개발 프로세스를 추적할 수 있습니다.😃

chenguokai 에 2020년 04월 06일

물론 가능하지만 develop 분기하고 이 분기를 계속 따르십시오. master 는 더 이상 빈번한 변경에 사용되지 않고 릴리스 및 핫픽스에만 사용됩니다.

Nightwalker-87 에 2020년 04월 06일

👍1

ILP32, LLP64 및 LP64 간의 호환성과 관련하여 데이터 유형 long 을 int32_t 로 교체하는 것이 좋은 아이디어라고 생각합니다. 그러나 미래에 다음과 관련된 놀라움을 배제하려면 변환, 우리는 가능하면 고정 너비 정수 유형으로 이동하는 것을 고려할 수 있습니다...

Nightwalker-87 에 2020년 05월 09일

👍1

@chenguokai : 이것은 당신에게 맡기겠습니다. 시간이 있고 마음에 든다면 기여할 수 있습니다.

Nightwalker-87 에 2020년 06월 03일

물론 출시 전에 할 것입니다. 지금 바쁘다.😔

chenguokai 에 2020년 06월 07일

👍1

이 페이지가 도움이 되었나요?

0 / 5 - 0 등급

Stlink: 통합 변수 유형으로 코드 정리

모든 6 댓글

관련 문제