Skip to content
This repository was archived by the owner on Oct 1, 2021. It is now read-only.

Commit a22fb53

Browse files
authored
Fix another Hue memory leak (#101)
1 parent f8aeafd commit a22fb53

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

netdisco/discoverables/philips_hue.py

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -29,14 +29,14 @@ def merge_entries(self, nupnp_entries, ssdp_entries):
2929
merges them making sure that same device is only discoverd once.
3030
3131
"""
32-
entries = nupnp_entries
32+
entries = {
33+
nupn_entry.description['URLBase']: nupn_entry
34+
for nupn_entry in nupnp_entries
35+
}
3336

34-
for nupn_entry in entries:
35-
for ssdp_entry in ssdp_entries:
36-
url_base1 = nupn_entry.description['URLBase']
37-
url_base2 = ssdp_entry.description['URLBase']
37+
for ssdp_entry in ssdp_entries:
38+
url_base = ssdp_entry.description['URLBase']
39+
if url_base not in entries:
40+
entries[url_base] = ssdp_entry
3841

39-
if url_base1 != url_base2:
40-
entries.append(ssdp_entry)
41-
42-
return entries
42+
return list(entries.values())

0 commit comments

Comments
 (0)