[Lcdproc] MtxOrb problem

[Ethan Dicks]

------=_Part_146996_1359251.1160072386785
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

On 10/6/06, Michelle Dupuis <support@ocg.ca> wrote:
> Sounds great - send over the test program when you have a chance.


> I noticed that the corrupt "]" character is at the end of a bar graph for a
> CPU that is at 2% utilization.  So, it's not a problem of the bar graph
> going too far to the right.  This looks more and more like a problem
> relating the number of display lines!

Entirely possible, but that's all client-dependent.

> What's strange is that without graphics, there were no scroll/overwrite
> issues.  So something to do with using the graphics characters appears to be
> causing a scroll / overwrite issue.  (The flickering reinforces that)

Are you sure you have the latest version of LCDproc? (some nightly
after 01-Oct)  I fixed a bug in MtxOrb.c that caused custom chars to
be displayed one line below and one char to the right from where they
should be (there was a missing 'x--' and 'y--' in MtxOrb_chr that
caused the accidental offset).  That could certainly cause all sorts
of overrun problems.

Here's the 2-line bargraph test client.  If you _are_ having
off-by-one problems with custom chars, this client won't look good
either, but it _is_ a 2-line client.

-ethan

------=_Part_146996_1359251.1160072386785
Content-Type: text/x-perl; name="lcdhbartest.pl"
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename="lcdhbartest.pl"
X-Attachment-Id: f_esxh4x7s

IyEvdXNyL2Jpbi9wZXJsIC13CgojICRJZDogbGNkaGJhcnRlc3QucGwsdiAxLjEgRXhwICQKCiMg
Rm9sbG93aW5nIGlzIHRoZSBQT0QgZG9jdW1lbnRhdGlvbiwgdHlwZSBwZXJsZG9jIGxjZGhiYXJ0
ZXN0LnBsIHRvIHJlYWQgaXQuCgo9aGVhZDEgTkFNRQoKbGNkaGJhcnRlc3QgLSBUZXN0IGhvcml6
b250YWwgYmFyIGNvZGUuCgo9aGVhZDEgU1lOT1BTSVMKCkI8bGNkaGJhcnRlc3QucGw+Cgo9aGVh
ZDEgREVTQ1JJUFRJT04KCkI8bGNkaGJhcnRlc3Q+IGlzIGFuIExDRHByb2MgY2xpZW50IHByb2dy
YW0gdGhhdCBkaXNwbGF5cyBvbmUgaG9yaXpvbnRhbApiYXIgb24gYSAyLWxpbmUgZGlzcGxheSwg
YW5kIHR3byBiYXJzIG9uIGEgNC1saW5lIGRpc3BsYXkuICBUaGUgZmlyc3QgYmFyCnJhbXBzIGZy
b20gMCUgdG8gMTAwJS4gIFRoZSBzZWNvbmQgYmFyIChpZiBwcmVzZW50KSByYW1wcyBmcm9tIDEw
MCUgdG8gMCUsCm1pcnJvcmluZyB0aGUgZmlyc3QgYmFyLgoKKy0tLS0tLS0tLS0tLS0tLS0tLS0t
KyAgKy0tLS0tLS0tLS0tLS0tLS0tLS0tKyAKfCMjIGhiYXIgeHh4JSAjIyMjIyNAfCAgfCMjIGhi
YXIgIHh4eCUgIyMjIyNAfAp8PCBob3Jpem9udGFsIGJhciAxID58ICB8PCBob3Jpem9udGFsIGJh
ciAxID58CnwgICAgICAgICAgICAgICAgICAgIHwgICstLS0tLS0tLS0tLS0tLS0tLS0tLSsKfDwg
aG9yaXpvbnRhbCBiYXIgMiA+fAorLS0tLS0tLS0tLS0tLS0tLS0tLS0rCgo9aGVhZDEgT1BUSU9O
UwoKPWhlYWQxIERJQUdOT1NUSUNTCgo9aXRlbSBDYW5ub3QgY29ubmVjdCB0byBMQ0Rwcm9jIHBv
cnQKCkJ5IGRlZmF1bHQsIGxjZGhiYXJ0ZXN0IHRyaWVzIHRvIGNvbm5lY3QgdG8gbG9jYWxob3N0
IHBvcnQgMTM2NjYuIElmIHlvdSBnZXQgdGhpcyAKZXJyb3IsIHRoYXQgbWVhbnMgdGhhdCB0aGlz
IGlzIG5vdCBwb3NzaWJsZSB0byBjb25uZWN0IHRvIHRoaXMgcG9ydC4gWW91IGNhbgpjaGFuZ2Ug
dGhlIHBvcnQgbGNkeG1tcyBjb25uZWN0cyB0byBieSBtb2RpZnlpbmcgdGhlIHNjcmlwdCdzIHZh
cmlhYmxlcyAkaG9zdAphbmQgJHBvcnQuCgo9aGVhZDEgUkVRVUlSRVMKClBlcmwgNS4wMDQ7CgpU
aGVzZSBhcmUgYWxsIGF2YWlsYWJsZSBvbiBDUEFOOiBodHRwOi8vd3d3LmNwYW4ub3JnLwoKPWhl
YWQxIERJU0NMQU1FUgoKVGhpcyBwcm9ncmFtIGlzIGZyZWUgc29mdHdhcmU7IHlvdSBjYW4gcmVk
aXN0cmlidXRlIGl0IGFuZC9vciBtb2RpZnkgCml0IHVuZGVyIHRoZSB0ZXJtcyBvZiB0aGUgR05V
IEdlbmVyYWwgUHVibGljIExpY2Vuc2UgdmVyc2lvbiAyIGFzIApwdWJsaXNoZWQgYnkgdGhlIEZy
ZWUgU29mdHdhcmUgRm91bmRhdGlvbi4KClRoaXMgcHJvZ3JhbSBpcyBkaXN0cmlidXRlZCBpbiB0
aGUgaG9wZSB0aGF0IGl0IHdpbGwgYmUgdXNlZnVsLCBidXQKV0lUSE9VVCBBTlkgV0FSUkFOVFk7
IHdpdGhvdXQgZXZlbiB0aGUgaW1wbGllZCB3YXJyYW50eSBvZiBNRVJDSEFOVEFCSUxJVFkgCm9y
IEZJVE5FU1MgRk9SIEEgUEFSVElDVUxBUiBQVVJQT1NFLiBTZWUgdGhlIEdOVSBHZW5lcmFsIFB1
YmxpYyBMaWNlbnNlIApmb3IgbW9yZSBkZXRhaWxzLgoKWW91IHNob3VsZCBoYXZlIHJlY2VpdmVk
IGEgY29weSBvZiB0aGUgR05VIEdlbmVyYWwgUHVibGljIExpY2Vuc2UgYWxvbmcgCndpdGggdGhp
cyBwcm9ncmFtOyBpZiBub3QsIHdyaXRlIHRvIHRoZSBGcmVlIFNvZnR3YXJlIEZvdW5kYXRpb24s
IEluYy4sIAo1OSBUZW1wbGUgUGxhY2UgLSBTdWl0ZSAzMzAsIEJvc3RvbiwgTUEgMDIxMTEtMTMw
NywgVVNBLgoKPWhlYWQxIEJVR1MKClllcywgdGhlcmUgbWlnaHQgYmUgc29tZS4gUGxlYXNlIHJl
cG9ydCBhbnkgeW91IGZpbmQgdG8gTENEcHJvYydzIG1haWxpbmcgbGlzdC4KU2VlIHRoZSB3ZWJz
aXRlIGZvciBtb3JlIGluZm9ybWF0aW9ucy4KCj1oZWFkMSBXRUJTSVRFCgpWaXNpdCBCPGh0dHA6
Ly9sY2Rwcm9jLm9tbmlwb3RlbnQubmV0Lz4gZm9yIG1vcmUgaW5mbyBhbmQgdGhlIGxhc3Rlc3Qg
dmVyc2lvbi4KCj1jdXQKCgoKdXNlIHN0cmljdDsKdXNlIElPOjpTb2NrZXQ7CnVzZSBGY250bDsK
CiMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMj
IyMjIwojIENvbmZpZ3VyYWJsZSBwYXJ0LiBTZXQgaXQgYWNjb3JkaW5nIHlvdXIgc2V0dXAuCiMj
IyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMj
IwoKIyBWZXJib3NlCiMgMCA6IE5vbmUgKG9ubHkgZmF0YWwgZXJyb3JzKQojIDEgOiBXYXJuaW5n
cwojIDUgOiBFeHBsYWluIGV2ZXJ5IHN0ZXAuCm15ICR2ZXJib3NlID0gMDsKCiMgSG9zdCB3aGlj
aCBydW5zIGxjZHByb2MgZGFlbW9uIChMQ0RkKQpteSAkaG9zdCA9ICJsb2NhbGhvc3QiOwoKIyBQ
b3J0IG9uIHdoaWNoIExDRGQgbGlzdGVucyB0byByZXF1ZXN0cwpteSAkcG9ydCA9ICIxMzY2NiI7
CgojIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMj
IyMjIyMKIyBFbmQgb2YgdXNlciBjb25maWd1cmFibGUgcGFydHMKIyMjIyMjIyMjIyMjIyMjIyMj
IyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjCiRTSUd7SU5UfSA9IFwm
Z3JhY2U7CiRTSUd7VEVSTX0gPSBcJmdyYWNlOwoKIyBDb25uZWN0IHRvIHRoZSBzZXJ2ZXIuLi4K
cHJpbnQgIkNvbm5lY3RpbmcgdG8gTENEcHJvYyBhdCAkaG9zdFxuIiBpZiAoJHZlcmJvc2UgPj0g
NSk7Cm15ICRyZW1vdGUgPSBJTzo6U29ja2V0OjpJTkVULT5uZXcoCiAgICAgICAgICAgICAgICBQ
cm90byAgICAgPT4gInRjcCIsCiAgICAgICAgICAgICAgICBQZWVyQWRkciAgPT4gJGhvc3QsCiAg
ICAgICAgICAgICAgICBQZWVyUG9ydCAgPT4gJHBvcnQsCiAgICAgICAgKSBvciBkaWUgIkNhbm5v
dCBjb25uZWN0IHRvIExDRHByb2MgcG9ydCAoJGhvc3Q6JHBvcnQpXG4iOwoKIyBNYWtlIHN1cmUg
b3VyIG1lc3NhZ2VzIGdldCB0aGVyZSByaWdodCBhd2F5CiRyZW1vdGUtPmF1dG9mbHVzaCgxKTsK
CnNsZWVwIDE7ICAgICAgICAjIEdpdmUgc2VydmVyIHBsZW50eSBvZiB0aW1lIHRvIG5vdGljZSB1
cy4uLgoKcHJpbnQgJHJlbW90ZSAiaGVsbG9cbiI7Cm15ICRsY2Rjb25uZWN0ID0gPCRyZW1vdGU+
OwpwcmludCAkbGNkY29ubmVjdCBpZiAoJHZlcmJvc2UgPj0gNSk7CiMgY29ubmVjdCBMQ0Rwcm9j
IDAuNC4yIHByb3RvY29sIDAuMyBsY2Qgd2lkIDIwIGhndCA0IGNlbGx3aWQgNSBjZWxsaGd0IDgK
KCRsY2Rjb25uZWN0ID1+IC9sY2QuK3dpZFxzKyhcZCspXHMraGd0XHMrKFxkKykvKTsKbXkgJGxj
ZHdpZHRoID0gJDE7IG15ICRsY2RoZWlnaHQ9ICQyOwpwcmludCAiRGV0ZWN0ZWQgTENEIHNpemUg
b2YgJGxjZHdpZHRoIHggJGxjZGhlaWdodFxuIiBpZiAoJHZlcmJvc2UgPj0gNSk7Cm15ICRoYmFy
d2lkdGggPSAkbGNkd2lkdGggLSAyOwoKIyBUdXJuIG9mZiBibG9ja2luZyBtb2RlLi4uCmZjbnRs
KCRyZW1vdGUsIEZfU0VURkwsIE9fTk9OQkxPQ0spOwoKIyBTZXQgdXAgc29tZSBzY3JlZW4gd2lk
Z2V0cy4uLgpwcmludCAkcmVtb3RlICJjbGllbnRfc2V0IG5hbWUge2xjZGhiYXJ0ZXN0fVxuIjsK
cHJpbnQgJHJlbW90ZSAic2NyZWVuX2FkZCBsY2RoYmFydGVzdFxuIjsKcHJpbnQgJHJlbW90ZSAi
c2NyZWVuX3NldCBsY2RoYmFydGVzdCBuYW1lIHtsY2RoYmFydGVzdH1cbiI7CgpwcmludCAkcmVt
b3RlICJ3aWRnZXRfYWRkIGxjZGhiYXJ0ZXN0IHRpdGxlIHRpdGxlXG4iOwpwcmludCAkcmVtb3Rl
ICJ3aWRnZXRfc2V0IGxjZGhiYXJ0ZXN0IHRpdGxlIHtoYmFyfVxuIjsKCnByaW50ICRyZW1vdGUg
IndpZGdldF9hZGQgbGNkaGJhcnRlc3QgdGV4dHVhbCBzdHJpbmdcbiI7CnByaW50ICRyZW1vdGUg
IndpZGdldF9hZGQgbGNkaGJhcnRlc3QgaGJhcjEgaGJhclxuIjsgICAgICMgc2F5IGhiYXIxIGlz
IGFuIGhiYXIKcHJpbnQgJHJlbW90ZSAid2lkZ2V0X3NldCBsY2RoYmFydGVzdCBoYmFyMSAxIDIg
MFxuIjsgICAgIyBwdXQgaXQgYXQgKDEsMikKCmlmICgkbGNkaGVpZ2h0ID4gMykgewogICAgcHJp
bnQgJHJlbW90ZSAid2lkZ2V0X2FkZCBsY2RoYmFydGVzdCBoYmFyMiBoYmFyXG4iOyAgIyBoYmFy
MiBpcyBhbiBoYmFyCiAgICBwcmludCAkcmVtb3RlICJ3aWRnZXRfc2V0IGxjZGhiYXJ0ZXN0IGhi
YXIxIDEgNCAwXG4iOyAjIHB1dCBpdCBhdCAoMSw0KQp9Cgp3aGlsZSAoMSkgewogICAgZm9yICht
eSAkcGN0ID0gMDsgJHBjdCA8PSAxMDAgOyAkcGN0KyspIHsKCglwcmludCAiaGJhciBsZW5ndGgg
aXMgbm93ICR7cGN0fSVcbiIgaWYgKCR2ZXJib3NlID49IDUpOwoKCSMgU2V0IHRleHQgbGluZSBm
aXJzdAoJbXkgJGNtZCA9IHNwcmludGYoIndpZGdldF9zZXQgbGNkaGJhcnRlc3QgdGl0bGUge2hi
YXIgJWQlJX1cbiIsICRwY3QpOwoJcHJpbnQgIkNtZCBpcyAnJGNtZCciIGlmICgkdmVyYm9zZSA+
PSA1KTsKCXByaW50ICRyZW1vdGUgJGNtZDsKCgkjIFNldCBoYmFyMSBsaW5lIG5leHQKCSRjbWQg
PSBzcHJpbnRmKCJ3aWRnZXRfc2V0IGxjZGhiYXJ0ZXN0IGhiYXIxIDEgMiAlZFxuIiwgJHBjdCk7
CglwcmludCAiQ21kIGlzICckY21kJyIgaWYgKCR2ZXJib3NlID49IDUpOwoJcHJpbnQgJHJlbW90
ZSAkY21kOwoKCSMgYW5kIHRoZSBvcHRpb25hbCBoYmFyMgoJaWYgKCRsY2RoZWlnaHQgPiAzKSB7
CgkgICAgJGNtZCA9IHNwcmludGYoIndpZGdldF9zZXQgbGNkaGJhcnRlc3QgaGJhcjIgMSA0ICVk
XG4iLCAxMDAgLSAkcGN0KTsKCSAgICBwcmludCAiQ21kIGlzICckY21kJyIgaWYgKCR2ZXJib3Nl
ID49IDUpOwoJICAgIHByaW50ICRyZW1vdGUgJGNtZDsKCX0KCglwcmludCAiU2xlZXBpbmcgZm9y
IDEgc2Vjb25kcy5cbiIgaWYgKCR2ZXJib3NlID49IDUpOwoJc2xlZXAgMTsKICAgIH0KCQogICAg
IyBlYXQgYWxsIGlucHV0IGZyb20gTENEZAogICAgd2hpbGUoZGVmaW5lZChteSAkaW5wdXQgPSA8
JHJlbW90ZT4pKSB7IH0KICAgIAp9CiMgU2hvdWxkIG5ldmVyIGdvIHRoZXJlIHNpbmNlIGFib3Zl
IGlzIGFuIGluZmluaXRlIGxvb3AsIAojIHNlbmQgYSBTSUdJTlQgb3IgU0lHVEVSTSB0byBleGl0
IG5pY2VseS4KCiMgVG8gYmUgY2FsbGVkIG9uIGV4aXQgYW5kIG9uIFNJR0lOVCBvciBTSUdURVJN
LgpzdWIgZ3JhY2UoKSB7CglwcmludCAiRXhpdGluZy4uLlxuIiBpZiAoJHZlcmJvc2UgPj0gNSk7
CgljbG9zZSgkcmVtb3RlKTsKCWV4aXQ7Cn0KCl9fRU5EX18K
------=_Part_146996_1359251.1160072386785--