for me the more readable form is to remove ret variable and use clear true, false in each situation like
```
if( ifcfg == nil) 
  return true;

if( bonded[ devname]:nil != nil)
{
   y2debug( "IsBridgeable: excluding lan item (%1: %2) - is bonded", itemId, devname); 
   return false;
}

if ( devtype == "br" )
  return false;

if ifcfg["STARTMODE"] == nfsroot"
  return false;

return true;