About the `_gettimeseq`, I'm too far from the code right now to provide youa
usefull answer, sorry. I might have better clues with a new topic patches about
that.
For the `file_use_mail_ts` name, I'd say it's ambigous. First, because `ts` is
an acronym. Second, because we don't know how the timestamp gets used. Perhaps
something like `filename_use_mail_timestamp`.
Overall it's nitpicking. What I do care is having a configuration option marked
experimental but the name by itself is not so important while it's not too much
exposed to name collision with further potential option names.
---
Reply to this email directly or view it on GitHub:
https://github.com/OfflineIMAP/offlineimap/pull/265#issuecomment-151916908
_______________________________________________
OfflineIMAP-project mailing list: [email protected]
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/offlineimap-project
OfflineIMAP homepages:
- https://github.com/OfflineIMAP
- http://offlineimap.org
