I think this is our first use of this GNU extension. I'm not opposed, but think it needs discussion before adding this dependency.
~~~~
671 + auto it = valid_displays.find(display);
672 + if (it == valid_displays.end())
673 + return;
674 +
675 + valid_displays.erase(it);
AFICS this is a verbose and inefficient
valid_displays.erase(display);
But I wouldn't block on it as it isn't a critical path.
~~~~
741 + throw new std::runtime_error{"Attempted to create Android client platform on non-Android server"};
1. Don't throw pointers
2. Use BOOST_THROW_EXCEPTION
~~~~
1067 + throw std::runtime_error{"Attempted to create Mesa client platform on non-Mesa server"};
...
1602 + throw std::logic_error{"Must call one of create_*_client_platform() before calling get_platform_library()"};
62 +#include <error.h> get_error_ message( surface) );
...
73 + error(1, 0, "Surface creation failed: %s\n", mir_surface_
I think this is our first use of this GNU extension. I'm not opposed, but think it needs discussion before adding this dependency.
~~~~
671 + auto it = valid_displays. find(display) ; end()) erase(it) ;
672 + if (it == valid_displays.
673 + return;
674 +
675 + valid_displays.
AFICS this is a verbose and inefficient
valid_ displays. erase(display) ;
But I wouldn't block on it as it isn't a critical path.
~~~~
741 + throw new std::runtime_ error{" Attempted to create Android client platform on non-Android server"};
1. Don't throw pointers EXCEPTION
2. Use BOOST_THROW_
~~~~
1067 + throw std::runtime_ error{" Attempted to create Mesa client platform on non-Mesa server"}; error{" Must call one of create_ *_client_ platform( ) before calling get_platform_ library( )"};
...
1602 + throw std::logic_
Use BOOST_THROW_ EXCEPTION