Workiva / go-datastructures

A collection of useful, performant, and threadsafe Go datastructures.
Apache License 2.0
7.66k stars 834 forks source link

Add nil check to Dtrie.Get() #207

Closed AliceBennett closed 4 years ago

AliceBennett commented 5 years ago

Dtrie.Get() currently panics if called with a key that does not exist (due to calling nil.Value() in this case). Adding a nil check here will prevent this panic from happening, and ensure that the behaviour is as described in documentation.

aviary-wf commented 5 years ago

Security Insights

No security relevant content was detected by automated scans.

Action Items

brianshannan-wf commented 4 years ago

@dustinhiatt-wf

asido commented 4 years ago

Fix tabs and spaces inconsistency.

teresarevious-wf commented 4 years ago

RM +1 dependencies verified when consumed in tier 1 repo