PDA

Archiv verlassen und diese Seite im Standarddesign anzeigen : [X Development] X0.80 fixes


Lucike
20.08.2003, 10:42
[equote:e2d38383fc="Verfasst am: Sa Jul 19.07.2003 20:05 von dvb.matt"]X080++
(1)
@X.java
@internal class: PREVIEW
@method: load(), line: ~2146

@fix: preview didn't jump properly among fileborders in case of more than 2 inputfiles

for (int a=0;a<previewList.size();a++){
...
if (startposition < end){
...
if (end-startposition < size && a<previewList.size()-1){
...
// }else *** old
} // *** new
break;
}
}
(2)
@X.java
@internal class: WORK
@method: vdrparse(), line: ~5014
@method: rawparse(), line: ~5616
@method: pvaparse(), line: ~6445

@fix: enabled cut- and split- pointer didn't work properly in case of more than 2 inputfiles

if (startPoint < 0)
...
else if (startPoint < count){
...
}
else if (startPoint > count){
//for (int a=1; a < starts.length; a++){ *** old
for (int a=FileNumber+1; a < starts.length; a++){ // *** new
if (starts[a] > startPoint)
break;
else
FileNumber++;
}
}[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: Sa Jul 19.07.2003 21:14 von TheHorse"]OK, schon nachgetragen. Obwohl ich ja meist nur einzelne Files bearbeite...
PS (OT): Hab mir den PVR 5000 geleistet. Dem ersten Eindruck nach hat der große Probleme mit Aufnahmen durch mein Walt Disney CAM.
So viele packets out of sequence hatte ich mit dem 4000 nie.[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: So Jul 20.07.2003 10:06 von dvb.matt"]So viele packets out of sequence hatte ich mit dem 4000 nie.
solangs nur auf unwichtigen PIDs ist..
ich entnehme dem aber, dass die rec wohl 'gestört' ist.[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: So Jul 20.07.2003 13:57 von dvb.matt"](3)
@MPVD.java
@internal class: Picture
@method: Add_Block(), line: ~2125++

@fix: remove cross coloured speckles in preview

if (cc==1){
for (int y=0;y<16;y++)
for (int x=0;x<16;x++){
val = chroma[x+(y*16)];
luma = pixels[(bx<<1)+x+(((by<<1)+y)*horizontal_size)];
r = (0xFF&luma>>>16);
// g = (0xFF&luma>>>8); //*** removed
// b = (0xFF&luma); // *** removed
g = (int)((double)(0xFF&luma>>>8) -0.34414*val); // *** changed
b = (int)((double)(0xFF&luma) +1.722*val); // *** changed
g = (g<0) ? 0 : (0xFF&g); // *** added
b = (b<0) ? 0 : (0xFF&b); // *** added
pixels[(bx<<1)+x+(((by<<1)+y)*horizontal_size)] = r<<16|g<<8|b;
}
}else if (cc==2){
for (int y=0;y<16;y++)
for (int x=0;x<16;x++){
val = chroma[x+(y*16)];
luma = pixels[(bx<<1)+x+(((by<<1)+y)*horizontal_size)];
b = (0xFF&luma);
// r = (0xFF&luma>>>16); // *** removed
// g = (0xFF&luma>>>8); // *** removed
r = (int)((double)(0xFF&luma>>>16) +1.402*val); // *** changed
g = (int)((double)(0xFF&luma>>>8) -0.71414*val); // *** changed
r = (r<0) ? 0 : (0xFF&r); // *** added
g = (g<0) ? 0 : (0xFF&g); // *** added
pixels[(bx<<1)+x+(((by<<1)+y)*horizontal_size)] = r<<16|g<<8|b;
}
}[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: So Jul 20.07.2003 19:55 von TheHorse"]So verbringst du den Sonntag...[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: Mo Jul 21.07.2003 15:50 von dvb.matt"]and're machen zu der Zeit 'n Mittagsschläfchen..
(ich war die Viertelstunde leider nicht müde )[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: Mo Jul 21.07.2003 19:56 von TheHorse"]Die Flecken sind weg.
Viertelstunde würde bei mir nicht reichen. Wenn ich mich hinleg, werden 4 Stunden draus.[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: Di Jul 22.07.2003 14:20 von chatweazle"]Hi Ihr!

Bin gerade "zufällig" hier vorbeigekommen und mich natürlich SOFORT angemeldet!

Wenn ich den Thread hier so durchlese, stellt sich mir die Frage ob ich die Fixes jetzt alle selbst eintragen und neu compilieren muß oder ob es von Zeit zu Zeit aktualisierte Quellen zum Download geben wird, die ich dann nur noch durch den Compiler jagen muß...
Java-Quelltext ist nämlich nicht gerade mein Sonntags-Hobby und da hätte ich doch schon ein wenig Angst, mehr kaputt zu machen als zu fixen

Gruß,

Chatweazle[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: Di Jul 22.07.2003 14:27 von TheHorse"]Iss nich so schlimm (zumindest das Abändern ). Musst halt nur auf die Klammern aufpassen .
BTW: Die Vorschau iss jetzt sehr schön. Läuft in Java natürlich ein bisschen nach, aber völlig praxistauglich.[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: Mi Jul 23.07.2003 7:43 von chatweazle"]Nachdem ich mir das JDK gesaugt habe, habe ich jetzt mal X.jar compiliert (ohne die Fixes) und bin jetzt schon begeistert!
Man spart sich das zwischenzeitliche konvertieren nach PVA um die Schnittpunkte bestimmen zu können!

Gruß,

Chatweazle[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: Mi Jul 23.07.2003 18:42 von dvb.matt"]@chatweazle

Fixes jetzt alle selbst eintragen und neu compilieren
yap, weil effektiver und nachvollziehbar für den/die Developer.
kaputt machen kann man dabei normal nichts.

die .jar kann im übrigen sonstwie heißen, X ist hier nur ein Pseudonym.

----
(4)
@AUDIO.java
@class: AUDIO
@method: ---, line: ~302

@fix: exception while parsing bad/wrong AC3-frames (bad index)

int ac3_bitrate_index[] = {
...
320000,384000,448000,512000,576000,640000,
// 0,0 // *** change to:
0,0,0,0,0,0,0,0,0,0,0,0,0 // *** new; arraysize should be 0x20
};[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: So Jul 27.07.2003 10:55 von Dr._Seltsam"]BTW: Die Vorschau iss jetzt sehr schön
Also, ich sehe keinen Unterschied. Das Bild enthält nach wie vor die gleichen hellgrün/blauen Flecken. Ist aber zum Bestimmen der Schnittpunkte völlig ausreichend.

@DVB.matt: Vielen Dank, dass Du uns Deine tolle Arbeit wieder zur Verfügung stellst. Nun brauche ich nicht mehr mit schlechtem Gewissen und heruntergezogenem Rollo ds.jar weiterbenutzen.

Gruß
Dr. Seltsam (ein tiefes Gefühl der Dankbarkeit empfindend)[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: So Jul 27.07.2003 11:28 von TheHorse"]die gleichen hellgrün/blauen Flecken.
Ja, einige Files habe ich jetzt auch wieder mit Flecken entdeckt. Aber die roten im schwarzen Hintergrund sind jetzt weg. Das macht aber nix, für das Handling ist das mehr als ausreichend.[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: So Jul 27.07.2003 15:01 von chatweazle"]Klingt einleuchtend (das WIE und WARUM des Selbst-Einpflegens der Fixes).
Ich hab' es auch gestern abend mal versucht, aber irgendwo hab' ich wohl was dabei verbockt... jedenfalls wollte mir das Kompilieren nicht so recht gelingen. Wahr wohl für die Konzentration schon ein Gals Rotwein zu viel!

Nun ja, werde es heute eben nochmal versuchen. Hab es ja in weiser Voraussicht nicht im "Original" gemacht.

Trotzdem: Wird es voraussichtlich in (un)regelmäßigen Abständen eine Veröffentlichung von X.81 etc. - Ständen geben? Wäre auch vorteilhaft bezüglich der Zeilenangaben für die Fixes...

Gruß,

Chatweazle

P.S.: Ich will damit auch in keinster Weise die Arbeit auf den Schöpfer der Ur-Version abwälzen. Dies soll viel mehr ein Aufruf an kompetente Mit-Entwickler sein...[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: So Jul 27.07.2003 17:43 von dvb.matt"]die gleichen hellgrün/blauen Flecken. Ist aber zum Bestimmen der Schnittpunkte völlig ausreichend.
beides stimmt...
( der Code ist eben "hausgemacht")

@chatweazle
der Compiler zeigt normalerweise die Zeilennummer bei Probs.
(von Interesse sind in den Posts nur die angemerkten *** Zeilen, alles andere bleibt so)

bei gravierenden Änderungen (posten ineffektiv) könnte (nur) die entspr. Datei erneuert werden.
So etwas steht aber vorerst nicht an...[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: Mo Jul 28.07.2003 19:02 von harbad1"]jetzt müsste ich nur mal wissen, was ich wo eintragen soll. hab von java prog. null ahnung.[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: Mo Jul 28.07.2003 19:43 von TheHorse"]was ich wo eintragen soll. hab von java prog. null ahnung.
Du meinst die Fixes? Also schöner beschreiben (mit Zeilenangabe) als matt es getan hat, kann man das wohl kaum.[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: Mo Jul 28.07.2003 20:33 von harbad1"]richtig die meine ich.
aber irgendwie steige ich da nicht durch.
nehmen wir mal fix1

for (int a=0;a<previewList.size();a++){
...
if (startposition < end){
...
if (end-startposition < size && a<previewList.size()-1){
...
// }else *** old
} // *** new
break;
}
}

ich nehm mir einen texteditor (editpad)
dann sprige ich in zeile 2146
dort steht:
:::::::::::::::::::::::::::::::::::::::::::::::::: ::}else
das ersetze ich durch:
}

(die doppelpunkte sollen leerzeichen sein, werden hier aber nichtangezeigt)

oder sehe ich das falsch? und was haben die // zu bedeuten?[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: Mo Jul 28.07.2003 20:41 von TheHorse"]dort steht: }else das ersetze ich durch: }
Genau so. // leiten Kommentare ein. Sonst würde es beim Kompilieren ganz schon hageln, wenn das dahinter Code wäre
Die Leerzeichen sind nicht so wichtig, das Einrücken soll nur den Code übersichtlicher machen.[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: Mo Jul 28.07.2003 21:26 von harbad1"]na da werd ich das mal probieren. was mach ich aber bei fix 4

0,0,0,0,0,0,0,0,0,0,0,0,0 // *** new; arraysize should be 0x20
};

jetzt hab ich die 2 nullen durch 13 nullen ersetzt und was mache ich mit // *** new; arraysize should be 0x20
};

muss ich da drunter zwei neue zeilen mit :

arraysize should be 0x20
};

einfügen?[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: Mo Jul 28.07.2003 21:39von dvb.matt"]was mache ich mit // *** ....
nichts besonderes, eben so übernehmen wie es da steht, oder weglassen.
ganz nach Lust und Laune.
(nicht groß drüber nachdenken, wenn du damit (Code) noch nichts am Hut hast)

theHorse hat's bereits erklärt: "// leiten Kommentare ein..."[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: Mo Jul 28.07.2003 22:04 von harbad1"]dann ist " // *** new; arraysize should be 0x20 " nur der kommentar für die nullen und " }; " bezieht sich schon auf die nächste zeile die schon da steht. jetzt könnte schon langsam der groschen gefallen sein.[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: Di Jul 29.07.2003 18:26 von dvb.matt"]der Fleckenteufel hat nochmal zugeschlagen:

(3a)
@MPVD.java
@internal class: Picture
@method: Add_Block(), line: ~2133, ~2145

@fix: remove a little bit more cross coloured speckles in preview

if (cc==1){
...
g = g<0 ? 0 : (g>255 ? 255 : g); // replacement of g = (g<0) ? 0 : (0xFF&g);
b = b<0 ? 0 : (b>255 ? 255 : b); // replacement of b = (b<0) ? 0 : (0xFF&b);
...
}else if (cc==2){
...
r = r<0 ? 0 : (r>255 ? 255 : r); // replacement of r = (r<0) ? 0 : (0xFF&r);
g = g<0 ? 0 : (g>255 ? 255 : g); // replacement of g = (g<0) ? 0 : (0xFF&g);
...
}[/equote:e2d38383fc]
[equote:e2d38383fc="Verfasst am: Di Jul 29.07.2003 21:46 von Dr._Seltsam"]der Fleckenteufel hat nochmal zugeschlagen:
...und er war gründlich. Hat diesmal ordentlich was gebracht, ich sehe keien Flecken mehr!
Gruß
Dr. Seltsam[/equote:e2d38383fc]