ImageLoader

[nberardi]

I have been looking over the internals of ImageLoader and curious what this code is suppose to do.

https://github.com/migueldeicaza/MonoTouch.Dialog/blob/master/MonoTouch.Dialog/Utilities/ImageLoader.cs#L317

if (uri == null){
    Console.Error.WriteLine ("Dropping request {0} because url is null", uri);
    pendingRequests.Remove (uri);
    uri = null;
}

The value of uri is already checked for null, so it can't be written to the console or removed from the pendingRequest.

This looks like just unneeded artifact code that is left over from a previous incarnation. Would you be willing to take a pull request to clean up this code and other things I have noticed in ImageLoader?