Optimize location lookup, avoid repeated config reads

jmathai / elodie

An EXIF-based photo assistant, organizer and workflow automation tool.

https://bit.ly/introducing-elodie

Apache License 2.0

1.27k stars 139 forks source link

Optimize location lookup, avoid repeated config reads #372

Open adamcandy opened 4 years ago

adamcandy commented 4 years ago

This caches the false states of 'prefer_english_names', which I expect is what was intended. Fixes #371 (Geolocation examines config.ini for every new lookup).

coveralls commented 4 years ago

Coverage decreased (-0.2%) to 90.877% when pulling aa023697d7246cb8e057ee637b9e44942dc2babb on adamcandy:prefer-english-names-optimize into 8e95b96e111ef1e892716a17aecfc9095fb52a90 on jmathai:master.

jmathai commented 4 years ago

This looks good. Can you cherry pick the commits from #370 into this branch so we can combine them into a single PR?