if this method were any longer than this, then i'd bitch about the early return here too ;-)
```
CPVRRecordingPtr retval;
PVRRecordingMap::const_iterator it = m_recordings.find(CPVRRecordingUid(iClientId, strRecordingId));
if (it != m_recordings.end())
  retval = it->second;
return retval;
````