Jump to content


Photo

Teletext improvements


  • Please log in to reply
104 replies to this topic

Re: Teletext improvements #21 littlesat

  • PLi® Core member
  • 53,259 posts

+619
Excellent

Posted 6 December 2013 - 22:16

I prefer to wait on pieterg's advice here....

WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W


Re: Teletext improvements #22 betacentauri

  • PLi® Core member
  • 7,074 posts

+314
Excellent

Posted 7 December 2013 - 22:38

the subtitle rendering is probably missing a

m_dc->islocked()

check.
Each painter / dc operation is checking the lock, it seems.
That's how the issue should have been fixed I think.

 

Hi pieterg,

 

I have debugged it and found the error:

In lib/gdi/grc.cpp in procedure blitScale I have added a eDebug which shows that m_dc->islocked() is always 0. Then I tried to find out which islocked method is called. It's not the gFBDC::islocked! It's the virtual islocked method in gDC (grc.h).

Ok, then I thought, the variable m_dc must be of type gDC, otherwise it should be called gFBDC::islocked. But at runtime typeid(m_dc).name() says "5gFBDC". So it's type is gFBDC. Why is then the virtual method called?

 

I found the answer. It's because the islocked method in gFBDC has a const qualifier:

int islocked() const {....};

After removing const it works. A little bit strange.

 

Here is a patch.

I guess with it Littlesat can revert his tuxtxt subtitle workaround (I have tested with the subtitles and it works without the patch)

Attached Files


Xtrend ET-9200, ET-8000, ET-10000, OpenPliPC on Ubuntu 12.04

Re: Teletext improvements #23 pieterg

  • PLi® Core member
  • 32,766 posts

+245
Excellent

Posted 7 December 2013 - 23:55

ok, good find.
Looks like it might have been working earlier this year

43ed3b0a (Mike Looijmans 2013-07-28 12:49:58 +0200 35) int islocked() const { return fb->islocked(); }

A reminder that we really should pay attention at the compiler warnings ;)
(and enable -Woverloaded-virtual, if we didn't so already)

Re: Teletext improvements #24 betacentauri

  • PLi® Core member
  • 7,074 posts

+314
Excellent

Posted 8 December 2013 - 10:33

Hi Pieterg, Milo:

I guess instead of removing const we can also add const to gDC::islocked. Or?
What do you prefer?
Xtrend ET-9200, ET-8000, ET-10000, OpenPliPC on Ubuntu 12.04

Re: Teletext improvements #25 pieterg

  • PLi® Core member
  • 32,766 posts

+245
Excellent

Posted 9 December 2013 - 09:40

yes, that would be better

Re: Teletext improvements #26 betacentauri

  • PLi® Core member
  • 7,074 posts

+314
Excellent

Posted 9 December 2013 - 20:15

Ok, here is the patch.

Attached Files


Xtrend ET-9200, ET-8000, ET-10000, OpenPliPC on Ubuntu 12.04

Re: Teletext improvements #27 betacentauri

  • PLi® Core member
  • 7,074 posts

+314
Excellent

Posted 11 December 2013 - 20:56

Ok, I see the patch is applied and littlesat has reverted the workarounds. I guess you get now problems applying my patches (http://openpli.org/f...ndpost&p=388940). I'll rewrite them in the next days and post them here again.


Xtrend ET-9200, ET-8000, ET-10000, OpenPliPC on Ubuntu 12.04

Re: Teletext improvements #28 littlesat

  • PLi® Core member
  • 53,259 posts

+619
Excellent

Posted 11 December 2013 - 22:00

Ok at least thanks!

WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W


Re: Teletext improvements #29 betacentauri

  • PLi® Core member
  • 7,074 posts

+314
Excellent

Posted 14 December 2013 - 21:37

Ok, here are the patches.

 

As said before: With the patch you can assign every teletext functionality to almost every button you like via keymap.xml

Attached Files


Xtrend ET-9200, ET-8000, ET-10000, OpenPliPC on Ubuntu 12.04

Re: Teletext improvements #30 quad

  • Member
  • 10 posts

0
Neutral

Posted 25 January 2014 - 20:36

I really like the idea of adding keymap.xml support to teletext (I would then add KEY_CHANNELUP and KEY_CHANNELDOWN for "page up" and "page down"). Unfortunately merging patches into files from the repository is exceeding my skills. I would really appreciate if this addition could be commited. What are the plans / schedule for that? Thanks..


Edited by quad, 25 January 2014 - 20:40.


Re: Teletext improvements #31 littlesat

  • PLi® Core member
  • 53,259 posts

+619
Excellent

Posted 25 January 2014 - 21:57

Isn't this still committed?

WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W


Re: Teletext improvements #32 betacentauri

  • PLi® Core member
  • 7,074 posts

+314
Excellent

Posted 26 January 2014 - 00:02

No, it's not committed. You wanted to wait what pieterg say to the patch.
http://openpli.org/f...ndpost&p=390836

Please post here a little info when you commit it!

Edited by betacentauri, 26 January 2014 - 00:03.

Xtrend ET-9200, ET-8000, ET-10000, OpenPliPC on Ubuntu 12.04

Re: Teletext improvements #33 littlesat

  • PLi® Core member
  • 53,259 posts

+619
Excellent

Posted 26 January 2014 - 15:45

I just tried... at this moment I'm not able to commit this patch...


WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W


Re: Teletext improvements #34 betacentauri

  • PLi® Core member
  • 7,074 posts

+314
Excellent

Posted 26 January 2014 - 15:58

Why? Error? Or other problem?
Xtrend ET-9200, ET-8000, ET-10000, OpenPliPC on Ubuntu 12.04

Re: Teletext improvements #35 littlesat

  • PLi® Core member
  • 53,259 posts

+619
Excellent

Posted 26 January 2014 - 16:02

$ git am -s ../tuxtxt_git_0001-Teletext-improvements.patch.txt
Applying: Teletext improvements
c:/######/tuxtxt/.git/rebase-apply/patch:212: trailing whitespace
.
#ifndef HAVE_TRIPLEDRAGON
error: patch failed: tuxtxt/plugin/plugin.py:16
error: tuxtxt/plugin/plugin.py: patch does not apply
Patch failed at 0001 Teletext improvements
The copy of the patch that failed is found in:
   c:/######/tuxtxt/.git/rebase-apply/patch
When you have resolved this problem, run "git am --resolved".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W


Re: Teletext improvements #36 betacentauri

  • PLi® Core member
  • 7,074 posts

+314
Excellent

Posted 26 January 2014 - 17:23

Patch is quite old. Perhaps something has changed. I'll take a look at it.
Xtrend ET-9200, ET-8000, ET-10000, OpenPliPC on Ubuntu 12.04

Re: Teletext improvements #37 betacentauri

  • PLi® Core member
  • 7,074 posts

+314
Excellent

Posted 26 January 2014 - 20:27

Strange! What did you do?

$ git clone git://git.code.sf.net/p/openpli/tuxtxt
Klone nach 'tuxtxt'...
remote: Counting objects: 263, done.
remote: Compressing objects: 100% (212/212), done.
remote: Total 263 (delta 129), reused 107 (delta 50)
Empfange Objekte: 100% (263/263), 173.72 KiB | 303.00 KiB/s, done.
Löse Unterschiede auf: 100% (129/129), done.
Prüfe Konnektivität... Fertig

$ cd tuxtxt
$ git am -s tuxtxt_git_0001-Teletext-improvements_v2.patch.txt 
Wende an: Teletext improvements
$

Important: Both patches must be applied! One in tuxtxt git and one in the e2 git.


Xtrend ET-9200, ET-8000, ET-10000, OpenPliPC on Ubuntu 12.04

Re: Teletext improvements #38 quad

  • Member
  • 10 posts

0
Neutral

Posted 5 February 2014 - 09:20

Betacentauri, are you saying that you are able to compile it correctly?

 

If so, should I send a message to pieterg asking him for a review and approval?



Re: Teletext improvements #39 betacentauri

  • PLi® Core member
  • 7,074 posts

+314
Excellent

Posted 5 February 2014 - 20:09

Hi quad,

 

that means that the patch could be applied without problems.

I didn't do another function test, but it should still work.

 

You can send pm to pieterg, but I guess he reads the forum and have no time to review it.


Xtrend ET-9200, ET-8000, ET-10000, OpenPliPC on Ubuntu 12.04

Re: Teletext improvements #40 littlesat

  • PLi® Core member
  • 53,259 posts

+619
Excellent

Posted 5 February 2014 - 21:08

I'll retry it later... Hold on...

WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W



0 user(s) are reading this topic

0 members, 0 guests, 0 anonymous users