search

ccebinger / SWPSoSe14

2 stars 1 forks source link

Bug: List append #220

Closed Zelldon closed 10 years ago

Zelldon commented 10 years ago

append Bytecode should get a conditional to check if its a list or not if not then create new list and these two objects into

kretzschi commented 10 years ago

Kannst du das Problem nochmal kurz mit einem Beispiel erklären? Kann irgendwie keinen Fehler reproduzieren. :)

Zelldon commented 10 years ago 
-----a--c--:---b---#
Zelldon commented 10 years ago

Wenn keine Liste aufm stack liegt funktioniert das nicht, deshalb checken ob liste da wenn nein dann liste erzeugen und beide Werte rein packen!

kretzschi commented 10 years ago

Okay, also: ----1---2:---# funktioniert jetzt. Daraus macht er: [1,2] Aber: ----n----1:--# funktioniert nicht, weil er ausgibt: [1,[]]

Das liegt daran, dass du im conditional branch nochmal add aufrufst und dadurch scheinbar die leere - gerade erstellte - Liste pushst.

Er soll ja nur im Fall von -----1---2:--# eine neue Liste erstellen und die beiden Werte reinpushen. Wenn schon eine Liste existiert -----n--1:--# dann brauch er die Liste auch nicht pushen.

jdettlaff commented 10 years ago

Habs mit #235 zusammen gefixt.