[PATCH] lcd patch for next release

Anders Eriksson <[email protected]> Mon, 06 Dec 2010 11:54:27 +0100
Newsgroups gmane.comp.video.freevo.devel
Message-ID <[email protected]>
Hi all,

In the spirit of shaking out all patches for the next release, I've got this one:

--- plugins/lcd.py.orig 2010-04-12 11:00:05.000000000 +0200
+++ plugins/lcd.py      2010-10-11 10:24:09.000000000 +0200
@@ -598,7 +598,7 @@
         # Animaton-Sequence used in audio playback
         # Some displays (like the CrytstalFontz) do display the \ as a /
         if self.version.startswith('0.5'):
-            self.animation_audioplayer_chars = ['-', '\\', '|', '/']
+            self.animation_audioplayer_chars = ['-', '\\\\', '|', '/']
         elif self.version.startswith('0.4'):
             self.animation_audioplayer_chars = ['-', '\\', '|', '/']
         else:


It ensure that backslashes are backshash prefixed on the wire when speaking to LCDd. 
If you don't add this, you get Tthis LCDd error:
Dec  4 20:58:35 tv LCDd: error: huh? Wrong number of arguments

The LCDd doc on the wireline format doesn explicitly say that backslashed needs to be
backslash prefixed, but reading the source shows that that need to be (and since 
backslashes are used as prefix, it's obvious it needs special treatment too).

I'd be happy to see this one applied.

-Anders


------------------------------------------------------------------------------
What happens now with your Lotus Notes apps - do you make another costly 
upgrade, or settle for being marooned without product support? Time to move
off Lotus Notes and onto the cloud with Force.com, apps are easier to build,
use, and manage than apps on traditional platforms. Sign up for the Lotus 
Notes Migration Kit to learn more. http://p.sf.net/sfu/salesforce-d2d