refactoring 제안

[ChoiWheatley]

• 매번 struct 붙이는 것이 짜증나지 않으신가요? 이번 기회에 한 번에 typedef로 타입을 짧게 쓰는 건 어떨까요? 전부 다 바꾸는 건 의미가 없으므로 자주 쓰일 타입들만 그렇게 작성해봅시다.

• struct thread ⟶ Thread

• struct list_elem ⟶ ListElem

• struct semaphore ⟶ Sema

• struct lock ⟶ Lock

[ChoiWheatley]

• 포매터 통일한 다음 언제한번 싹 돌리고 싶네요

[ChoiWheatley]

swjungle-week07-09/threads/synch.c

Lines 210 to 213 in 5c7c1fa

	} else if (waiter_max->priority < cur->priority) { // 첫 빠따가 아닌 경우
	// 기존의 max priority가 현재 thread의 priority보다 작은 경우 기존 d_elem 제거 후 새로 추가
	list_remove(&waiter_max->d_elem);
	list_insert_ordered(dlist, &cur->d_elem, origin_priority_dsc_d, NULL);

• lock_acquire 실패할 경우 기부자 목록에 넣을 때 정렬해서 넣을 필요가 없답니다

[coding-jjun]

• get_thread_elem -> elem_to_thread으로 refactoring하기
• get_thread_d_elem -> d_elem_to_thread으로 refactoring하기