INTERNAL: Bundle nested fields of prefix_t with NESTED_PREFIX

[6unYoung]

🔗 Related Issue

• https://github.com/issues/assigned?issue=jam2in%7Carcus-works%7C704

⌨️ What I did

• prefix_t의 parent_prefix, child_prefix_items 필드를 용도에 맞게 NESTED_PREFIX 태그로 묶었습니다.
• NESTED_PREFIX 태그가 정의되었을 때만 해당 변수들이 사용되게 수정했습니다.

[ing-eoking]

만약 DEFAULT_PREFIX_MAX_DEPTH 값이 1보다 큰 경우 Non-nested prefix 상황에서도 delimeter를 기준으로 prefix가 여러 개 생성되어 prefix_list에 각각 담기게 됩니다.

Non Nested 구조일 때는 1개의 prefix만 담도록 할 수 있나요?

[6unYoung]

네 가능합니다. prefix_list 생성 시 NESTED_PREFIX가 정의되어 있지 않다면 break하도록 하면 될 듯합니다.

별개로 NESTED_PREFIX를 정의하고 DEFAULT_PREFIX_MAX_DEPTH가 5로 설정 후 key를 0:1:2:3:4:5를 주는 경우에, 각 레벨별 prefix의 생성은 확인됩니다. 하지만 stats prefixes나 stats prefixlist item명령을 입력했을 때 0까지만 출력되는 문제가 있습니다.
아마도 stats prefixes/prefixlist에 NESTED_PREFIX를 고려하지 않아 발생한 문제라 이 부분도 같이 수정해야 할 것 같습니다.

[ing-eoking]

Child Prefix에 대한 출력은 의도적으로 제외한 것으로 보입니다.

if (pt->parent_prefix != NULL) { /* skip child prefixes */
  pt = pt->h_next;
  continue;
}

다만, 현재 Nested Prefix 기능은 아직 완전히 구현된 상태가 아니므로,
지금 시점에서는 NESTED_PREFIX 관련 부분까지 수정할 필요는 없어 보입니다.

[ing-eoking]

현재로서는 Non-Nested Prefix 로직에서 prefix_list, prefix_depth, DEFAULT_PREFIX_MAX_DEPTH를 사용하고 있는 부분이 조금 부자연스러워 보입니다. 이 부분을 Non Nested Prefix와 Nested Prefix를 활용할 때를 분리할 수 있나요?

[6unYoung]

현재 NESTED_PREFIX를 사용하지 않는 경우에도 prefix_list[0]을 사용하고 있습니다. 이 변수를 사용하지 않는다면 prefix_link()에서 두 경우의 로직 자체를 분리해야 할텐데, 가독성이 떨어지지 않을까요?