hibiscus icon indicating copy to clipboard operation
hibiscus copied to clipboard

remove redundant local variable

Open ruderphilipp opened this issue 3 years ago • 2 comments

Entfernt doppelt erzeugte Referenzen auf ein identisches Objekt. Diese stellt nur ein Alias dar und hat keinen Mehrwert, wenn direkt darunter bereits schon wieder eine Zuweisung/ ein return erfolgt.

ruderphilipp avatar Nov 28 '21 17:11 ruderphilipp

Ehrlichweise würde ich diesen PR ungemerged schliessen. Die Änderungen bringen jetzt nun nicht den riesen Vorteil, erzeugen aber an zwei potentiellen Stellen Probleme und erzeugt derzeit einen Merge-Konflikt. Das ist doch schade um unser aller Zeit.

willuhn avatar Jan 18 '22 06:01 willuhn

Ehrlichweise würde ich diesen PR ungemerged schliessen. Die Änderungen bringen jetzt nun nicht den riesen Vorteil, erzeugen aber an zwei potentiellen Stellen Probleme und erzeugt derzeit einen Merge-Konflikt. Das ist doch schade um unser aller Zeit.

Änderungen passen aus meiner Sicht. (Insgesamt stimme ich aber zu, dass die Änderungen kaum etwas bewirken werden, zumal die meisten Stellen bisher bestimmt sogar vom Compiler rausoptimiert wurden.) Merge-Konflikt ist nur ein Datei-Konflikt, innerhalb der Dateien aber konfliktfrei.

meigelb avatar Jan 26 '22 00:01 meigelb