[SENTRY-1659] HMSPathsDumper has useless code

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Won't Fix
• Component/s: Sentry
• Labels: bite-sized newbie
• Assignee: Krishna Kalyan
• Reporter: Alex Kolbasov
• Created: 15/Mar/17 06:19
• Updated: 25/Oct/17 18:52
• Resolved: 30/May/17 21:59

Description

The HMSPathsDumper class has the following code:

      if (child.getAuthzObjs().size() != 0) {
        for (String authzObj: child.getAuthzObjs()) {
          Set<Entry> paths = authzObjToPath.get(authzObj);
          if (paths == null) {
            paths = new HashSet<Entry>();
          }
          paths.add(child);
        }
      }

Note that local var paths is created inside the loop and never used, so this piece of code is completely useless.

Issue Links

• is duplicated by: SENTRY-1752 HMSFollower gets stuck once it fails to process a notification event