Skip to content

ConfigParser 리펙토링 & 린트룰 수정#30

Open
Two-Jay wants to merge 7 commits intodevfrom
jekim_#29
Open

ConfigParser 리펙토링 & 린트룰 수정#30
Two-Jay wants to merge 7 commits intodevfrom
jekim_#29

Conversation

@Two-Jay
Copy link
Contributor

@Two-Jay Two-Jay commented Apr 20, 2022

  • 파싱 값 저장 구조체를 std::map<std::string, std::string> 에서 std::map<std::string, std::vectorstd::string>으로 수정
  • cpp린트 기준으로 코드 포멧 수정 : 라인 80 줄이랑 레퍼런스->포인터 사용권고는 수정 x
  • 로케이션 블록을 임의로 파싱하여 서버블록 파싱값과 같이 저장하도록 수정
  • 자잘한 에러 수정

KIM JEONGJUN added 5 commits April 19, 2022 17:36
테스트함수 전용 네임스페이스 추가
필요성이 없어진 이유로 함수의 특정 시그니쳐를 삭제합니다.
테스트로그 함수 업로드, server block도 볼 수 있도록 임시 지역변수 생성
짜잘한 버그를 수정하고 린트룰에 맞추고 아침늦잠을 잤습니다....
@Two-Jay Two-Jay linked an issue Apr 20, 2022 that may be closed by this pull request
@Two-Jay Two-Jay requested review from Jiwon-Woo, kyoshong and sma96 April 20, 2022 11:14
KIM JEONGJUN added 2 commits April 20, 2022 22:02
불필요해진 파일 삭제
에러 없는 버전으로 다시 올라갑니다.
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.

[TEST] ConfigParser 테스트 코드 생성

1 participant