-
Notifications
You must be signed in to change notification settings - Fork 2
[FEAT/#157] 사진 촬영 완료 후 EditPhoto 화면으로 이동 시 비디오 캡쳐를 중단합니다. #159
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
모듈 맵은 헤더 파일을 가져오는 데 걸리는 시간을 단축하여 소스 컴파일 시간을 향상시킵니다. 모듈 맵은 컴파일러에게 프레임워크에 포함된 헤더 목록을 제공합니다. 하지만 저희 프레임워크들을 모듈 맵이 따로 없어 전부 NO로 설정했습니다.
hsw1920
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 이제 EditPhoto에서 아이폰이 불타지 않겠군요..!
|
|
||
| public func stopCaptureLocalVideo() -> Bool { | ||
| guard let capturer = self.videoCapturer as? RTCCameraVideoCapturer else { return false } | ||
| capturer.stopCapture() | ||
| return true | ||
| } | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
캬 바로 반영해주셨네요!
0Hooni
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이제 내 폰 안뜨거워진다 ㅠㅠ
수고 많으셨습니다 👍
| CODE_SIGN_STYLE = Automatic; | ||
| CURRENT_PROJECT_VERSION = 1; | ||
| DEFINES_MODULE = YES; | ||
| DEFINES_MODULE = NO; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
무슨 수정인걸까요 궁금궁금
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
저도 궁금합니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
프레임워크를 외부에 배포할 때 헤더를 제공하기 위해 모듈 맵이라는 걸 사용하는데, 이 모듈맵을 사용할건지를 체크하는 옵션이라고 해요. 저희 프레임워크들은 전부 내부 모듈이라 모듈 맵이 따로 없어 전부 NO로 설정했습니다.
Kiyoung-Kim-57
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
빠른 적용 감사함다!
🤔 배경
사진 편집화면부턴 서로의 영상을 보지 않기 때문에 불필요한 리소스였고, CPU와 메모리를 많이 사용하는 앱인 만큼 최적화가 필요했습니다.
📃 작업 내역
✅ 리뷰 노트
PhotoRoomViewController 와 EditPhotoRoomViewController 간에 연결이 되어있지 않아 동작은 확인하지 못했습니다..!