Comment improvements

Common/GPU/OpenGL/GLFeatures.cpp

@@ -135,7 +135,7 @@ bool CheckGLExtensions() {
 		return true;
 	}
 
-	memset(&gl_extensions, 0, sizeof(gl_extensions));
+	gl_extensions = {};
 	gl_extensions.IsCoreContext = useCoreContext;
 
 	const char *renderer = (const char *)glGetString(GL_RENDERER);

Common/GPU/OpenGL/GLFeatures.h

@@ -30,7 +30,7 @@ enum {
 // Extensions to look at using:
 // GL_NV_map_buffer_range (same as GL_ARB_map_buffer_range ?)
 
-// WARNING: This gets memset-d - so no strings please
+// WARNING: This gets memset-d - so no strings or other non-POD types please
 // TODO: Rename this GLFeatures or something.
 struct GLExtensions {
 	int ver[3];

android/jni/app-android.cpp

@@ -655,7 +655,7 @@ extern "C" void Java_org_ppsspp_ppsspp_NativeApp_init
 	EARLY_LOG("NativeApp.init() -- begin");
 	PROFILE_INIT();
 
-	std::lock_guard<std::mutex> guard(renderLock);
+	std::lock_guard<std::mutex> guard(renderLock);  // Note: This is held for the rest of this function - intended?
 	renderer_inited = false;
 	androidVersion = jAndroidVersion;
 	deviceType = jdeviceType;
@@ -875,6 +875,7 @@ extern "C" void Java_org_ppsspp_ppsspp_NativeApp_shutdown(JNIEnv *, jclass) {
 }
 
 // JavaEGL. This doesn't get called on the Vulkan path.
+// This gets called from onSurfaceCreated.
 extern "C" bool Java_org_ppsspp_ppsspp_NativeRenderer_displayInit(JNIEnv * env, jobject obj) {
 	_assert_(useCPUThread);
 
@@ -884,7 +885,10 @@ extern "C" bool Java_org_ppsspp_ppsspp_NativeRenderer_displayInit(JNIEnv * env,
 	// We should be running on the render thread here.
 	std::string errorMessage;
 	if (renderer_inited) {
-		// Would be really nice if we could get something on the GL thread immediately when shutting down.
+		// Would be really nice if we could get something on the GL thread immediately when shutting down,
+		// but the only mechanism for handling lost devices seems to be that onSurfaceCreated is called again,
+		// which ends up calling displayInit.
+
 		INFO_LOG(G3D, "NativeApp.displayInit() restoring");
 		EmuThreadStop("displayInit");
 		graphicsContext->BeginAndroidShutdown();