This logic confused me for a bit. Wouldn't it be easier to understand if it where
```
if (m_egl->GetPreferredResolution(&fallback))
  resolutions.push_back(fallback);
else
{
  CLog::Log(LOGERROR, "%s: Fatal Error, GetPreferredResolution failed",__FUNCTION__);
  return;
}
```
i.e. first handle the positive case and then the negative case?

If you want to keep it that way I'd suggest to at least remove the "else" because it's not necessary thanks to the early return in the "if" block.