[split] rcore, rcore_web and rcore_desktop changes (batch 2)
#3334
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 5 commits
September 20, 2023 22:54
ghost
changed the title
rcore_web.c changes (batch 2)rcore, rcore_web and rcore_desktop changes (batch 2)
|
@raysan5 On batch 3 I'm removing all uneeded platform macros from |
|
@ubkp Great work! Thank you very much! Merging it! |
ghost
deleted the
|
@raysan5 Thank you! 👍 |
ghost
mentioned this pull request
raysan5
added a commit
that referenced
this pull request
Oct 8, 2023
* Submodules (#3311) * Check in current state * Add submodules to Makefile and clean up some imports * Start moving InitGraphicsDeivce * Move android_main and CloseWindow() out of rcore * Move WindowShouldClose out of rcore * Move IsWindowHidden out of rcore * Move IsWindowMinimized out of rcore * Move IsWindowMaximized, IsWindowFocused and IsWindowResized out of rcore * Move ToggleFullscreen out of rcore * Move MaximizeWindow, MinimizeWindow and RestoreWindow out of rcore * Move 13 functions out of rcore: ToggleBorderlessWindowed SetWindowState ClearWindowState SetWindowIcon SetWindowIcons SetWindowTitle SetWindowPosition SetWindowMonitor SetWindowMinSize SetWindowMaxSize SetWindowSize SetWindowOpacity SetWindowFocused * Minor clean up, revert makefile change, include submodules directly in rcore * Fix makefile comment * Remove rcore.h from Makefile * Remove debug include * Move 18 functions from rcore to submodules GetWindowHandle GetMonitorCount GetCurrentMonitor GetMonitorPosition GetMonitorWidth GetMonitorHeight GetMonitorPhysicalHeight GetMonitorRefreshRate GetWindowPosition GetWindowScaleDPI GetMonitorName SetClipboardText GetClipboardText ShowCursor HideCursor EnableCursor DisableCursor GetTime * Move TakeScreenshot, OpenURL, GetGamepadName out of rcore into submodules * remove debugging #defines * Move GetMonitorPhysicalWidth from rcore to submodule * Move GetGamepadAxisCount from rcore * Move SetGamepadMappings out of rcore * Move GetMouseX, GetMouseY, GetMousePosition out of rcore * Move SetMousePosition out of rcore * Move GetMouseWheelMove out of rcore * Move the last functions out of rcore * Move shared function defs and some global var to rcore.h * Clean up rcore.c and rcore.h a little more * Remove unnecessary #define --------- Co-authored-by: MichaelFiber <michael@cubeofb.org> * REVIEWED: `PLATFORM_DESKTOP` Windows building * Revert "REVIEWED: `PLATFORM_DESKTOP` Windows building" This reverts commit 71a1217. * Reviewed Windows building * [split] Fix compilation for web (and desktop) (#3329) * Fix compilation for web * Remove EM_ASM_INT from core_input_gestures_web example * Fix raymath undefined symbols for desktop and web * Remove raylib_opengl_interop from examples Makefile * Revert previous commit (8651c78) * Fix TraceLog for web and desktop * [split] `rcore`, `rcore_web` and `rcore_desktop` changes (batch 2) (#3334) * Fix formatting * Reapply commit 9d230d7 (#3305) that was missing * Reapplies commits 719365f (#3309) and 8a1779b (#3312) that were missing * Reapply commit 5c9cc3f (#3323) that was missing * Reapply commit a2b3b1e that was missing * Revert commit cef25c6 to fix macro redefined warning * Move rcore.h #include to after config.h to fix macro redefinitions warnings * [split] `rcore`, `web`, `desktop`, `android` changes (batch 3) (#3338) * First pass to remove unneeded platform macros for web * Second pass to remove unneeded platform macros for web * Move GetTouchX, GetTouchY, GetTouchPosition from rcore to web, desktop, android * Move SetMouseCursor from rcore to android, desktop, web * [split] `rcore`, `web`, `desktop`, `android` changes (batch 4) (#3343) * Fix ToggleBorderlessWindowed duplicated glfwSetWindowSize calls * First pass to remove unneeded platform macros for android * Second pass to remove unneeded platform macros for android * Remove unneeded platform macros for desktop * Relocate GetGamepadName and update SetGamepadMappings on android, desktop, web * Add missing comment to web * [split] `rcore`, `web`, `desktop`, `android` changes (batch 5) (#3345) * Move SetExitKey from core to android, desktop, web * Move some callbacks from core to desktop and web * Relocate emscripten callbacks on web * Relocate android callbacks on android * Revert "Relocate android callbacks on android" This reverts commit bbdbecc. * Updates UnloadVrStereoConfig on rcore * Update SetClipboardText on android * Fix screenMin/Max default values for android * [split] `rcore`, `drm` changes (#3347) * Tweak makefiles for PLATFORM_DRM and move rcore_drm's dependencies to rcore.h * Move drm functions to rcore_drm.c * Fix a typo in rcore.c * Add SetExitKey to rcore_drm.c --------- Co-authored-by: MichaelFiber <michael@cubeofb.org> * Fix compilation for android (#3360) * Fix android include (#3364) * Reviewed platform split #3313 - Added file headers info - Added TRACELOG message for unimplemented functions - Reviewed code formatting and organization - Several code tweaks * REVIEWED: `GetDirectoryPath()` --------- Co-authored-by: MichaelFiber <42419558+michaelfiber@users.noreply.github.com> Co-authored-by: MichaelFiber <michael@cubeofb.org> Co-authored-by: ubkp <118854183+ubkp@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR changes
Fixes formatting on
rcore_web.cso it's easier to compare with current master branchrcore.c.On
rcore_web.c: R86, R107, R212, R226, R262, R292-R293, R302-R305, R321, R380, R382-R383, R385-R386, R388-R389, R392, R395, R397-R398, R400-R401, R405-R406, R418, R421-R422, R493, R511-R512, R537-R538, R791, R794, R797, R1059, R1448, R1473.Reapplies commit 9d230d7 (Implement FLAG_WINDOW_RESIZABLE for web #3305) that was missing for
PLATFORM_WEB.On
rcore_web.c: R162, R164, R205-R206, R216-R217.Reapplies commits 719365f (Add SetWindowMaxSize for desktop and web #3309) and 8a1779b (Rename windowM* to screenM* #3312) that were missing for both
PLATFORM_WEBandPLATFORM_DESKTOP.On
rcore.h: R146-R147.On
rcore_web.c: R219-R223, R344-R348, R1292-R1293, R1301-R1302.On
rcore_desktop.c: R192-R196, R1096-R1101, R1108-R1113.Reapplies commit 5c9cc3f ([rcore] Duplicated code on
EventThread()#3323) that was missing forPLATFORM_DESKTOP.On
rcore.c: R3369-R3373.Reapplies commit a2b3b1e that was missing for all platforms.
On
rcore.c: R1694-R1695, R1697-R1699.Revert my previous commit cef25c6 to fix the
warning: 'SUPPORT_TRACELOG' macro redefined(I forgot it was already defined onconfig.h, sorry).On
rcore.h: L10.Moves
#include "rcore.h"to after#include "config.h"onrcore.cto fix themacro redefinedwarnings. This way the definitions onconfig.hget carried torcore.h.On
rcore.c: L102, R108.Status
PLATFORM_WEB,PLATFORM_DESKTOPstatus #3339Environment
emscripten/emsdkon Linux (Mint 21.1 64-bit).Edit 1: added line marks.
Edit 2, 3, 4, 5: added missing commits.
Edit 6: updated status and notes.
Edit 7: revert a previous commit.
Edit 8: added line marks and editting.
Edit 9: added macro redefinition fix.
Edit 10: moved status to separate issue.