[wishlist] twitter/identi.ca operation could be more bandwidth friendly

This would be a handy feature! Not only for bandwidth but also because if you now get a bad connection between an interval gwibber stops showing all of the previous messages.

experimental branch attached, please test!

I get twitter retrieve responses errors in that build and it doesn't show any tweets. (it does show identi.ca and jaiky posts)
-"TypeError: 'NoneType' object must be a strong or a number and not a 'NoneType'" (shows this error right from the start when you start gwibber)
-"TypeError: 'NoneType' object is not iterable" (shows this after the first refresh and it keeps on giving this one everytime it refreshes)

that's strange.. i'm running it right now and it is doing fine

... although i'm not yet happy with performance, probably the list subscripting is performing slowly

or actually more likely the 200 iterations of the addMessages javascript

I got it working by changing the value of messages (the global one) from None to "" in twitter.py.

Thanks, tidied up based on your feedback. Hopefully should be more robust now.

I tried the tidied up version on my laptop and it works! And on the desktop it was still working without any of the problems the regular gwibber bzr version has, nice work!

Apparently something breaks when receive_count is None:
evan@~/src/Net/Twitter/bug-311793]» ./run
/home/evan/src/Net/Twitter/bug-311793/gwibber/microblog/support/facelib.py:47: DeprecationWarning: the md5 module is d
  import md5
20 new tweets since last_message_id -1
Traceback (most recent call last):
  File "/home/evan/src/Net/Twitter/bug-311793/gwibber/microblog/twitter.py", line 161, in receive
    for data in self.get_messages():
  File "/home/evan/src/Net/Twitter/bug-311793/gwibber/microblog/twitter.py", line 127, in get_messages
    messages = (new_messages + (messages or []))[0:int(self.account["receive_count"]) or 20]
TypeError: int() argument must be a string or a number, not 'NoneType'

It looks like a typo int(None) vs int(None or 20). The attached patch fixes it for me.

Thanks for the patch. Accepted and pushed up to branch.

p.s. have merged latest from lp:gwibber into this branch, note that ./run has been removed now and for testing branches the same functionality is now found in ./bin/gwibber (i.e., adding local files to python sys.path)

When will we see that in Gwibber?

The since_id value is now used for Twitter, Identi.ca, and Facebook.

Fixed as per last comment in 2.29.91