-
Type: Improvement
-
Resolution: Unresolved
-
Priority: Unknown
-
None
-
Affects Version/s: None
-
Component/s: None
I'm migrating my app (100% Kotlin) to Realm 4.0 which seems to have made OrderedRealmCollectionImpl.get and Collection.Iterator.get nullable. It seems like the only reason for this is because of the following lines in OrderedRealmCollectionImpl.get:
if (forValues) { // TODO implement this return null; }
forValues is final and set to false. I now have hundreds of call points where I have to take nullability into account (I don't like using !! unless I absolutely need to) because of something that will never return null.
Does this actually have to be nullable, or is it really a placeholder for something else? Because that would be a really inconvenient placeholder.