C-toxcore: .h 파일에 OS별 헤더를 포함하지 마십시오.

좋은 소리

"Windows 시스템"과 "posix 시스템" 및 "테스트 시스템"으로 구현할 수 있는 "시스템" 인터페이스를 만드는 것은 어떻습니까? 테스트 시스템은 단위 테스트에 사용될 수 있으며 실제로 시스템 호출을 수행하지 않고 시스템을 시뮬레이션합니다.

나는 구현 제안에 동의하지 않습니다. 각 플랫폼에 대해 다르게 작성해야 하는 함수를 래핑하는 데 #ifdef 가 사용되는 단일 파일 {network,time,...}.c 모든 함수를 훨씬 더 보고 싶습니다.

개별 파일에 비해 이 방법의 이점은 모든 플랫폼 특정 코드를 정적 함수로 강제 적용하고 더 높은 수준의 toxcore에 대해 단일 API를 노출할 수 있다는 것입니다. 모든 플랫폼 구현 세부 정보는 단일 파일에서 쉽게 볼 수 있습니다. 플랫폼 특성은 추상화와 여러 파일이 아닌 단일 파일 내에서 처리해야 합니다. 또한 cmakelists.txt 파일을 유지 관리하기 쉽게 만들 것입니다(이미 우리에게서 멀어지기 시작했습니다.). 그리고 현재 검토 시스템으로 나중에 검토합니다.

완전히 고정되지 않음

@GrayHatter가 동의했습니다. OS 사양을 구현하는 단일 파일을 만들어 보겠습니다.

지금은 @Diadlo 를 할당 해제합니다. 나는 이것이 많은 코드베이스와 네트워크 코드에 노출되기 때문에 새로운 기여자들이 작업할 수 있는 훌륭한 버그라고 생각합니다. 기능에 영향을 미치지 않는 다소 흥미로운 리팩토링이므로 테스트하기가 더 쉽습니다.