michalbe / jsnews-client

:alien: Commandline client for JS News
http://github.com/michalbe/jsnews-client
MIT License
38 stars 5 forks source link

Obsługa cache #73

Closed leriel closed 10 years ago

leriel commented 10 years ago

Poniżej mój pomysł na rozwiązanie issue #33. Użyłem do tego eksperymentalnie LevelDB dzięki czemu mam minimalnie granularny dostęp do cache, które mogę zapisywać niezależnie.

Dodałem też obsługę zakończenia aplikacji w funkcji cleanOnExit - obecnie nie ma potrzeby jej użycia, ale podejrzewam że przy dalszym użyciu bazy cache, zapotrzebowanie na nią pojawi się bardzo szybko

W dodatku fix dwóch bugów które wykryłem, komentarze dodam w liniach:

michalbe commented 10 years ago

Kapitalne, dzięki! Travis green, nie mam zastrzeżeń więc ląduje.