Fix merging DateComponents not working

davedelong / time

Robust and type-safe date and time calculations for Swift

MIT License

2.32k stars 77 forks source link

Fix merging DateComponents not working #43

Closed tapsandswipes closed 4 years ago

tapsandswipes commented 4 years ago

We expect DateComponents.value(for:) to return nil for non existant componets but it return NSNotFound, making marging components return only the other components, not the merge.

Signed-off-by: Antonio Cabezuelo Vivo antonio@tapsandswipes.com

davedelong commented 4 years ago

Great find, thank you!