-
Notifications
You must be signed in to change notification settings - Fork 56
Refactoring to remove global state #170
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
publixsubfan
wants to merge
40
commits into
master
Choose a base branch
from
glvis-window-dev
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
40 commits
Select commit
Hold shift + click to select a range
6c00bda
Move background setting to MyExpose
publixsubfan a1c18c4
Move antialiasing to aux_vis
publixsubfan dd9b278
Create GLVisWindow class, move font handling stuff into it
publixsubfan 403282f
Moving more methods to GLVisWindow class
publixsubfan a392026
Move idle func handling into GLVisWindow
publixsubfan d40b414
Add mouse events, model spinning stuff to GLVisWindow
publixsubfan 9ff3eeb
Move MSAA, screenshot controls into GLVisWindow, add wrapper for memb…
publixsubfan 1dfc47a
Move screenshot implementation to new file
publixsubfan e3cc12a
Move some window control functions to GLVisWindow
publixsubfan e7b9806
Move more scene transformation methods into GLVisWindow
publixsubfan 93c95f8
Fix a segfault with font ptr in VisualizationScene
publixsubfan d836dde
Remove GetUseTexture/SetUseTexture functions, move into aux_js
publixsubfan 69072c1
Move thread control functions into GLVisWindow
publixsubfan 84815a3
Remove some global dependencies from glvis.cpp
publixsubfan 2707c46
Remove references to global glvis_command instance
publixsubfan e369884
Move command, vis scene construction into GLVisWindow; pass local GLV…
publixsubfan dfe6aca
JS-related build fixes
publixsubfan 7c4b7a0
Remove global vsdata dependence from Plane class
publixsubfan 15a2a74
Start moving over some key events to member functions
publixsubfan eb59307
Add support for stackable events, improve handling of scene view resets
publixsubfan a8dcb86
Add support for conditionally-updatable event handlers
publixsubfan ca18aa2
Make all vssolution3d event handlers member functions
publixsubfan 664b7f1
Make vssolution event handlers member functions
publixsubfan bfb252c
Make vsvector event handlers member functions
publixsubfan 8100b38
Convert last of vsdata event handlers
publixsubfan 4b6f7b7
Remove SdlWindow reference from vsdata
publixsubfan 2e9032d
Fix for printing
publixsubfan 24485b3
Cleanup globals from aux_vis
publixsubfan 53e7370
Make TouchPinch a member of GLVisWindow
publixsubfan 9990835
Run astyle
publixsubfan 1e4c54e
Remove more globals
publixsubfan 942c632
Remove global stream_state from desktop side code
publixsubfan 4c97a29
Merge branch 'master' into glvis-window-dev
publixsubfan 2bb63be
Remove unused variable
publixsubfan 5e023b8
fix js build
51f67e8
fix some bugs & emscripten incompatibilities
44fd6d8
'fix' issue where the canvas size doubles on new vis; not sure why th…
c0de293
make style
tzanio 9ba13ce
Fix for vsvector grid function initialization
publixsubfan 31cc55b
Simplify GLVisCommand constructors
publixsubfan File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
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
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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.