Refactor TizenRenderers

swift-kim commented 3 years ago

Common

Merge InitializeRenderer and DestroyRenderer into constructors and destructors.
Inline GetWindowHandle.
Code cleanups.

Ecore Wl2

Combine SetupDisplay and SetupEcoreWlWindow into a single function SetupEcoreWl2.
Combine SetupEglWindow and SetupEglSurface into a single function SetupEGL.
Remove unnecessary private methods.
Add additional error descriptions for EGL errors.

Evas GL

Let SetupEvasWindow return bool to make the calling order consistent with Ecore Wl2.

bwikbs commented 3 years ago

The refactoring is neat and great! :+1: And.. What motivated you to do this? I wonder what's bothering you :smile:

swift-kim commented 3 years ago

@bwikbs I was writing TizenRendererWayland (just for fun), and wanted to make the code style consistent across the renderers. :)

bwikbs commented 3 years ago

@swift-kim You really hate efl!!! :smile:

flutter-tizen / engine