CyanogenMod: Bugs und Cherry-Picks

[MadMurdoc]

Fühlt euch frei euch darüber zu unterhalten.

 

[thE_29]

Facebook ist auch so eine App, die beim Starten GPS abfragt und danach laut Akku-Anzeige den Strom leer saugt..

Sehr sehr ärgerlich.. Auf was warten die eigentlich solange beim Mergen?

 

[Der_Schubi]

Naja, einige der Anpassungen aus dem Patch-Set sind etwas unsauber und umstritten. Da wird halt an denen weiter gebaut während die funktionierenden unangetastet rumliegen und alt werden.

Steve Kondik Dec 30 12:00 PM

Patch Set 1: Do not submit

BREAKS ALL THE THINGS

Unless you have a device with QCOM's JB stack (like D2), then this will probably help.

[...]

Needs ifdefs and whatnot to not break everyone else.

 

[rauke]

Mein ROM ist Stock CM + 3 Zeilen Code für den GPS Patch

Welcher GPS-Patch?

 

[thE_29]

Naja, dann sollen sie halt die C-Sourcen unberührt lassen.. Die 3 Zeilen waren im Java-Code und das ist simpelstes Java (ich bin Java-Entwickler)!! Einfach, wenn die size() = 0, dann stoppen..

Das zerstört rein gar nichts auf keinem Gerät.. Ifdefs & Co sind C/C++ und nicht in Java..

 

[bugz]

Naja, dann sollen sie halt die C-Sourcen unberührt lassen.. Die 3 Zeilen waren im Java-Code und das ist simpelstes Java (ich bin Java-Entwickler)!! Einfach, wenn die size() = 0, dann stoppen..

Das zerstört rein gar nichts auf keinem Gerät.. Ifdefs & Co sind C/C++ und nicht in Java..

Die Frage ist, warum das dann nicht implementiert wird... Hört sich für mich nach einer Zeile an!

 

[thE_29]

Die Frage ist, warum das dann nicht implementiert wird... Hört sich für mich nach einer Zeile an!

Kommentar, if und Ausführung = 3 Zeilen
Gerrit Code Review

Was man beachten sollte: CM ist ja nicht nur fürs N4, sondern für viel mehr Devices, weswegen mergen nicht so ruckzuck geht..
Warum sie aber für so simple Changes aber meistens warten, ist halt für außenstehende (also wir) schwer nachzuvollziehen..

Es ist halt ärgerlich.. :/

 

[Der_Schubi]

@bugz:
Link zu meinem Post

@Rauke:
der Bug:
Link zu meinem Post

 

[rauke]

Komisch, dass dieses Problem bei mir nicht auftritt.

 

[bugz]

Stimmt den ternären Operator gibts auch noch... xD

if (globalRecords.isEmpty()) applyRequirementsLocked(this.mProvider);

Für eine einfach if-Abfrage finde ich das ok, wenn mehr im Block steht, dann wird es etwas unübersichtlich.

PS: c:geo funktioniert bei mir problemlos.

 

[MadMurdoc]

Eröffnet.

 

[_mocus]

ingress ist auch so eine bei mir..

die starte ich, nichtmal eine Minute später wird das Gerät warm bis heiß... und frisst entsprechend akku wie ein heizpilz.

Das war bei stock rom nicht so.

 

[Der_Schubi]

Bleibt danach das GPS in der Akku Verbrauchsanzeige als an dargestellt, obwohl es aus ist?

Gesendet mit Tapatalk 2

 

[_mocus]

Bleibt danach das GPS in der Akku Verbrauchsanzeige als an dargestellt, obwohl es aus ist?

Gesendet mit Tapatalk 2

Wo genau sehe ich das?

Edit. Gefunden.

Scheint mit der cm10.1 vom 10.1. Behoben.

 

[MadMurdoc]

Versucht bitte Bilder direkt ins Forum als Anhang hochzuladen oder irgendwo anders. Die Bilder sind ja so groß das sie schon stören.

 

[_mocus]

Versucht bitte Bilder direkt ins Forum als Anhang hochzuladen oder irgendwo anders. Die Bilder sind ja so groß das sie schon stören.

Nö in tapatalk sind die schön eingepasst.

 

[Der_Schubi]

Hm, das GPS hat sich brav abgeschaltet. Alles OK auf dem Screenshot...

 

[LordN]

Kann noch jemand bestätigen dass der "GPS schaltet sich nicht aus"-Bug verschwunden ist seit gestern?

 

[Der_Schubi]

Bei welchem ROM? Bei den offiziellen nightlies ist er noch nicht behoben.

Gesendet mit Tapatalk 2

 

[LordN]

Meinte die offiziellen...