Skip to content
This repository has been archived by the owner on Dec 1, 2024. It is now read-only.

fix that default pkg's method count is at most 1 #47

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Tao93
Copy link

@Tao93 Tao93 commented Apr 4, 2017

Bug description: No matter how many default package methods there are, the printed result is always 1 for default package methods count. Explanation: The name in line 49 may be "", which means corresponding method is from default package. For the default package methods, code runs into else branch since no child has name "". Therefore, for every default package method, a new Node is created in line 53. For non-first default package method, the newly created Node will always replace the former Node in line 59, and get its count equals 1 in line 63. Overall, method count of default package is always at most 1. I added an else if branch, which avoids making new Node if we already has a Node when the coming method is a default package method.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant