Schwarzer Bildschirm nach Startbildschirm

klawitter schrieb:

Warning hierzu: Potential leak of an object allocated on line 960 and stored into 'addAnnotation'

Wenn Du eine init(...)-Methode aufrufst, bekommst Du laut Konvention bereits ein für Dich retaintes Objekt. Bei "init" kann das retain entfallen.

klawitter schrieb:

line 176 MKReverseGeocoder *geocoder = [[MKReverseGeocoder alloc] initWithCoordinate:newLocation.coordinate];
geocoder.delegate = self;
[geocoder start];

Vermutlich: Du speicherst lokal "geocoder" und weißt nach dem Verlassen nicht mehr, welches Objekt das war. Wie willst Du es dann releasen? So etwas gehört in eine Instanzvariable (ggf. als @private) oder in eine globale Variable (unschön).

klawitter schrieb:

Value stored to 'row' during its initialization is never read
--> unkritisch ?

Ja. Kannst Dir überlegen, ob Du die Variable überhaupt brauchst, sonst raus nehmen. Das macht der Compiler zwar auch automatisch, hält Deinen Code aber schlank.

klawitter schrieb:

NSFileManager *fileManager = [NSFileManager defaultManager];
.....
[fileManager release];

Don't do that! Du holst Dir ein Singleton und sendest ihm ein release. Gehört Dir das Singleton? Nein. Kannst Du ihm dann ein release schicken? Nein. Kannst Du ihm ein retain schicken? Nein, denn es ist ein Singleton, existiert nur einmal und wird mit Sicherheit nicht von Dir verwaltet. Also: Singletons nur nutzen und nicht retainen.

klawitter schrieb:

Warning hierzu: Potential leak of an object allocated on line 960 and stored into 'addAnnotation'

Ein retain zu viel und damit ein release zu wenig (s.o.)

#EDIT: Verwende bitte code-Tags!