[uBasic] "Nested if" Bug

CHDK-Skripte, CHDK-Entwicklung, PC-Zusatzprogramme, Informationen für Tüftler

Beitragvon msl » 26.09.2009, 13:48

Hallo,

das Kompilieren ist kein Problem. Da können ich und andere bei Bedarf auch helfen.

Ich würde jetzt aber mal die Reaktion im internationalen Forum abwarten. Wenn der Patch kurzfristig umgesetzt werden sollte, werden wir in aller Ruhe Skripte und Dokumentation ändern. Der ein oder andere Benutzer der Skripte wird sich schon melden, wenn etwas klemmt. Dann kann allemal reagiert werden.

Bisher hatten es die meisten CHDK-Nutzer hier im Forum beim Testen auch nicht eilig. ;)

Gruß msl
Benutzeravatar
msl
Super-Mod
Super-Mod
 
Beiträge: 4567
Bilder: 271
Registriert: 22.02.2008, 11:47
Wohnort: Leipzig
Kamera(s): A720 1.00c
SX220 1.01a

Beitragvon CHDKLover » 27.09.2009, 08:28

Hallo,
ich habe heute morgen gesehen, dass sich reyalp des Themas annehmen angenommen hat. Er wird es jetzt selber noch ausgiebig testen. Da können wir zur Zeit nur warten.

CHDKLover
A610 100e CHDK-DE: aktuelle Version
Benutzeravatar
CHDKLover
Super-Mod
Super-Mod
 
Beiträge: 878
Bilder: 8
Registriert: 12.09.2007, 18:25
Wohnort: Dresden
Kamera(s): a610 100e

Beitragvon CHDKLover » 04.10.2009, 15:39

Hallo,
jetzt steht der Patch kurz vor dem Einzug in den trunk. Wir brauchen aber noch so viel Skripte wie nur möglich, die Fehler im Syntax aufweisen. Zur Unterstützung habe ich ein kleines Testprogramm geschrieben, welches versucht fehlerhafte Scripte zu identifizieren.
Es währe super wenn ihr alle uBasic Scripte testen könntet, die ihr findet.

Anwendung des Programms:
Alle Skripte müssen in ein Verzeichnis eurer Wahl kopiert werden. Danach downloadet ihr das Archiv im Anhang, entpackt es in das selbe Verzeichnis. Jetzt muss nur noch die start.bat ausgeführt werden und das Testergebnis findet man unter "erg.txt".

Danke, CHDKLover
Dateianhänge
tester.zip
Programm + start.bat
(5.17 KiB) 751-mal heruntergeladen
erg.txt
Meine ersten Scripte im Test
(1.13 KiB) 768-mal heruntergeladen
A610 100e CHDK-DE: aktuelle Version
Benutzeravatar
CHDKLover
Super-Mod
Super-Mod
 
Beiträge: 878
Bilder: 8
Registriert: 12.09.2007, 18:25
Wohnort: Dresden
Kamera(s): a610 100e

Beitragvon rudi » 04.10.2009, 16:35

Hallo CHDKLover,

du hast ja gute Neuigkeiten. Da habe ich doch gleich alle Scripte gesucht, aber es sind nur eigenen Skripte zum Testen und die von gehtnix natürlich. Siehe Anhang.

Dein Tester ist ja Klasse, dieser zeigt wohl faule IF-Konstruktionen? So etwas sollte man für GOTOs in Verbindung mit Schleifen und mehrzeilige Befehle haben um Stackprobleme erkennen zu können. Oder man passt eben etwas auf.

Grüße Rudi.
Dateianhänge
erg.txt
(2.9 KiB) 733-mal heruntergeladen
Benutzeravatar
rudi
CHDK-Spezialist
CHDK-Spezialist
 
Beiträge: 510
Registriert: 11.09.2009, 11:27
Kamera(s): A590IS_101B, SX260_100B

Beitragvon gehtnix » 04.10.2009, 16:55

Hi CHDKLover,

das hast Du fein gemacht.
MD-Six und Sevenup sind bei mir die Hauptübelskripte.
Vor allem MD-SIX, aber das war ja die Mutter des IF-Problems.

erg.txt ergibt bei mir schon 0 Fehler

gruß gehnix
Benutzeravatar
gehtnix
CHDK-Legende
CHDK-Legende
 
Beiträge: 2406
Bilder: 8
Registriert: 17.04.2008, 12:42
Wohnort: München
Kamera(s): A610 100e+f + IXUS990 IS

Beitragvon msl » 04.10.2009, 19:58

Geniales Tool. =D>

Nun können wir nur hoffen, dass der Patch endlich eingepflegt wird. Dann ist es ein Kinderspiel, betroffene Skripte zu korrigieren.

Mittlerweile kennen wir die wichtigsten Kandidaten, bei denen Änderungen vorzunehmen sind.

Gruß msl
Benutzeravatar
msl
Super-Mod
Super-Mod
 
Beiträge: 4567
Bilder: 271
Registriert: 22.02.2008, 11:47
Wohnort: Leipzig
Kamera(s): A720 1.00c
SX220 1.01a

Beitragvon gehtnix » 04.10.2009, 20:43

msl hat geschrieben:Mittlerweile kennen wir die wichtigsten Kandidaten, bei denen Änderungen vorzunehmen sind.
:evil:

CHDKLover präsentiert noch Links im Englischen, alle auf meine Skipte :evil:

Und fudgey präsentiert auch noch einen klassischen IF-Bug.

Aber wenn es der Sache dienlich ist, warum nicht :D

gruß gehtnix
Benutzeravatar
gehtnix
CHDK-Legende
CHDK-Legende
 
Beiträge: 2406
Bilder: 8
Registriert: 17.04.2008, 12:42
Wohnort: München
Kamera(s): A610 100e+f + IXUS990 IS

Beitragvon CHDKLover » 07.10.2009, 06:59

gehtnix hat geschrieben:CHDKLover präsentiert noch Links im Englischen, alle auf meine Skipte :evil:

Und fudgey präsentiert auch noch einen klassischen IF-Bug.

Aber wenn es der Sache dienlich ist, warum nicht :D

Ja gehtnix,
es geht hierbei nicht um die Verfasser der Skripte, sonder nur um den zusätzlichen Aufwand nach der Patcheinführung abschätzen zu können. Es ist doch gut das wir wissen, dass diese 2 besagten Skripte zu bearbeiten sind, damit wir schnell eine lauffähige Version raus bringen können. Nicht das es erst auffällt, wenn schon etliche user verzweifelt das Skript als nicht funktionierend gelöscht haben.

CHDKLover
A610 100e CHDK-DE: aktuelle Version
Benutzeravatar
CHDKLover
Super-Mod
Super-Mod
 
Beiträge: 878
Bilder: 8
Registriert: 12.09.2007, 18:25
Wohnort: Dresden
Kamera(s): a610 100e

Beitragvon msl » 13.10.2009, 21:54

Hallo,

scheinbar haben wir keine Lobby im internationalen Forum. Es ist gegenwärtig unmöglich, diejenigen zu überzeugen, die die Berechtigung haben, Änderungen einzuchecken.

Vielleicht liegt es auch daran, das es zu wenig Fürsprache in dem Thread im int. Forum gibt.

Gruß msl
Benutzeravatar
msl
Super-Mod
Super-Mod
 
Beiträge: 4567
Bilder: 271
Registriert: 22.02.2008, 11:47
Wohnort: Leipzig
Kamera(s): A720 1.00c
SX220 1.01a

Beitragvon rudi » 15.10.2009, 21:58

Hallo

Ja, ich finde es auch bedauerlich, dass sich niemand dazu durchringen kann das Patch dem trunk hinzuzufügen.

Dann könnte man ja einen Stichtag, Vorschlag 3.10. (oder 10.10), festlegen.
Und gehtnix war so optimistisch. Oder hat er die Jahreszahl absichtlich weggelassen?

Gruß Rudi
Benutzeravatar
rudi
CHDK-Spezialist
CHDK-Spezialist
 
Beiträge: 510
Registriert: 11.09.2009, 11:27
Kamera(s): A590IS_101B, SX260_100B

Beitragvon CHDKLover » 16.10.2009, 07:43

Hallo rudi,
du hast Recht, dies demotiviert jeden, der am CHDK-Projekt mitarbeiten möchte auch mich. Seit PhyrePhoX nicht mehr so aktiv dabei ist, ist es auch für gute Patches schwierig in das CHDK aufgenommen zu werden.
Ich werde mich aber weiterhin um Besserung bemühen.

CHDKLover
A610 100e CHDK-DE: aktuelle Version
Benutzeravatar
CHDKLover
Super-Mod
Super-Mod
 
Beiträge: 878
Bilder: 8
Registriert: 12.09.2007, 18:25
Wohnort: Dresden
Kamera(s): a610 100e

Beitragvon gehtnix » 19.10.2009, 15:00

Hallo,

dass der Patch immer noch nicht im Trunc ist sehe ich inzwischen als offenen Affront gegenüber allen CHDK-Nutzern an!

Gerade diese rem-Konstrukt, da kommt kein normaler Basic-Anwender drauf. Dass ich das im Md-Six schon vor einem Jahr so gemacht habe war reiner Zufall! Hinter dem rem stand zunächst für mich ein Hinweis. Als ich dieser dann überflüssig war ging nix mehr. Erst jetzt, durch rudi, kam die Erklärung und Lösung.

Der Hinweis bei setepontos, LUA hätte das nested-if Problem nicht, zeigt schon den Grad der Bereitschaft auf den Mangel zu beseitigen.

Was mich stutzig macht. Hier bei wiklemms haben 2585 Usern fast 11000 Zugriffe zu diesem Thema. Bei seteponots mit 6316 Usern gerade mal 412 Zugriffe. Da drängt sich mir die Frage auf, passiert da bei setepontos noch was?

Auf zu neuen Servern!

geschmeidig bleiben

gruss gehtnix Bild
Benutzeravatar
gehtnix
CHDK-Legende
CHDK-Legende
 
Beiträge: 2406
Bilder: 8
Registriert: 17.04.2008, 12:42
Wohnort: München
Kamera(s): A610 100e+f + IXUS990 IS

Beitragvon rudi » 04.12.2009, 08:24

Hallo,

nun ist unser Patch auch im int. trunk (siehe Changeset 854).

Auch wurden die neuen Stack-Fehlermeldungen hinzugefügt.

@msl: Danke für die Zusammenstellung der Patchdatei.

Gruß rudi
Benutzeravatar
rudi
CHDK-Spezialist
CHDK-Spezialist
 
Beiträge: 510
Registriert: 11.09.2009, 11:27
Kamera(s): A590IS_101B, SX260_100B

Beitragvon msl » 04.12.2009, 15:22

Ja das wurde auch Zeit.

Nun fragt reyalp sogar nach weiteren geprüften Veränderungen/Korrekturen/Neugigkeiten.
Mal sehen, was wir ihm aus unserer DE-Version noch verkaufen können. Bild


Gruß msl
Benutzeravatar
msl
Super-Mod
Super-Mod
 
Beiträge: 4567
Bilder: 271
Registriert: 22.02.2008, 11:47
Wohnort: Leipzig
Kamera(s): A720 1.00c
SX220 1.01a

Beitragvon naddel » 04.12.2009, 23:43

Hallo Tüftler,

Ich habe mir gerade das ganze Thema durchgelesen und finde es
großartig
wie ihr das hinbekommen habt und wie da Eines ins Andere gegriffen hat.
Man findet sich allzuoft mit unnötigen Unzulänglichkeiten ab.
IHR nicht.

Gratuliere
naddel
S2 1.00f mit aktueller DE Version
Benutzeravatar
naddel
CHDK-Begeisterter
CHDK-Begeisterter
 
Beiträge: 156
Registriert: 26.01.2009, 19:42
Kamera(s): G3 s2 ixusii

VorherigeNächste

Zurück zu Code-Ecke

Wer ist online?

Mitglieder in diesem Forum: 0 Mitglieder und 2 Gäste

cron