Trying to get a Genpix working with VDR.
I thought my troubles started when Reinhard posted his new vdr-1.5.12-h264-syncearly-framespersec-audioindexer-fielddetection.diff cause I got compile errors, (should of known better).
But after digging back thru all the old patches I could find I believe this has been wrong for awhile.
What is/was wrong was that the ‘switch (frontendType)‘ function did not open and close the ‘{‘ ‘}’ properly (thankz Reinhard for pointing it out) after the patch.
The original code is like this
switch (frontendType) {
case FE_QPSK: { // DVB-S
The patched code looks like this
switch (frontendType) {
case FE_QPSK: // DVB-S
case FE_DVB_S: // DVB-S
case FE_DVB_S2: // DVB-S
As you can see the last ‘{‘ is missing and a ‘}’ had tobe removed later down in the code to make things happy. That is why I wonder how the patch has been working in the past.
I tried to add the ‘{‘ in after the S2 and leave the removed 1 in but it wouldn’t compile, (figured that be too easy).
What I have done is put the original ‘case FE_QPSK: { // DVB-S’ routine back into dvbdevice.c and have added a new ‘case FE_DVB_S: { // DVB-S’ that mirrors ‘case FE_QPSK: { // DVB-S’ routine and made a new ‘case FE_DVB_S2: { // DVB-S’ routine under the switch.
I am no coder by any means but can cut and paste with the best of them. It does seem to patch and compile fine but really unsure if what I did will be ok.
Was wondering if someone with a little more knowledge than me could take a look @ the patch and see if it all looks ok.
Thankz
No virus found in this outgoing message. Checked by AVG Free Edition. Version: 7.5.503 / Virus Database: 269.16.9/1155 - Release Date: 11/27/2007 8:30 PM