String utilities for stem

Hi Ravi, just pushed some new string utilities (mostly stolen from arm) to my 'str_tools' branch...

https://gitweb.torproject.org/user/atagar/stem.git/shortlog/refs/heads/str_tools

added component::archived/stem owner::neena priority::medium resolution::implemented status::closed type::enhancement labels

Trac:
Status: new to needs_review

Looks good.

+  try:
+    time_sum = int(float(seconds))
+    time_sum += int(minutes) * 60
+    time_sum += int(hours) * 3600
+    time_sum += int(days) * 86400
+    return time_sum

If you don't mind importing datetime, that could be replaced with...

return datetime.timedelta(days=int(days), hours=int(hours), minutes=int(minutes), seconds=int(float(seconds))).total_seconds()

That is all.

+SIZE_UNITS_BYTES = (
+  (1125899906842624.0, " PB", " Petabyte"),
+  (1099511627776.0, " TB", " Terabyte"),
+  (1073741824.0, " GB", " Gigabyte"),
+  (1048576.0, " MB", " Megabyte"),
+  (1024.0, " KB", " Kilobyte"),
+  (1.0, " B", " Byte"),
+)

On OS X 10.6+, 1 kb = 1000 bytes, not 1024 bytes. Not sure if it'll break anything. Just, FYI - https://support.apple.com/kb/TS2419

If you don't mind importing datetime, that could be replaced with...

Oh, neat. I didn't know that you could do that. I don't mind importing datetime, but personally I find the present summation to be a little bit easier to read than constructing a throwaway datetime object. But that's certainly debatable.

On OS X 10.6+, 1 kb = 1000 bytes, not 1024 bytes. Not sure if it'll break anything. Just, FYI

Thanks for the heads up. This won't break anything - this is simply a function for getting human readable labels and, frankly speaking, OSX is wrong about this.

Trac:
Status: needs_review to closed
Resolution: N/A to implemented

closed