Age | Commit message (Collapse) | Author | |
---|---|---|---|
2016-06-30 | DRTVWR-418: Unify control flow through LLAppViewer across platforms. | Nat Goodspeed | |
The LLApp API used to consist of init(), mainLoop(), cleanup() methods. This makes sense -- but on Mac that structure was being subverted. The method called mainLoop() was in fact being called once per frame. There was initialization code in the method, which (on Mac) needed to be skipped with an already-initialized bool. There was a 'while' loop which (on Mac) needed to be turned into an 'if' instead so the method would return after every frame. Rename LLApp::mainLoop() to frame(). Propagate through subclasses LLAppViewer and LLCrashLogger. Document the fact that frame() returns true to mean "done." (This was always the case, but had to be inferred from the code.) Rename the Mac Objective-C function mainLoop to oneFrame. Rename the C++ free function it calls from runMainLoop() to pumpMainLoop(). Add comments to llappdelegate-objc.mm explaining (inferred) control flow. Change the Linux viewer main() and the Windows viewer WINMAIN() from a single LLAppViewer::mainLoop() call to repeatedly call frame() until it returns true. Move initialization code from the top of LLAppViewer::frame() to the init() method, where it more properly belongs. Remove corresponding mMainLoopInitialized flag (and all references) from LLAppViewer. Remove 'while (! LLApp::isExiting())' (or on Mac, 'if (! LLApp::isExiting())') from LLAppViewer::frame() -- thus unindenting the whole body of the 'while' and causing many lines of apparent change. (Apologies to reviewers.) There are four LLApp states: APP_STATUS_RUNNING, APP_STATUS_QUITTING, APP_STATUS_STOPPED and APP_STATUS_ERROR. Change LLAppViewer::frame() return value from (isExiting()) (QUITTING or ERROR) to (! isRunning()). I do not know under what circumstances the state might transition to STOPPED during a frame() call, but I'm quite sure that if it does, we don't want to call frame() again. We only want a subsequent call if the state is RUNNING. Also rename mainLoop() method in LLCrashLogger subclasses LLCrashLoggerWindows, LLCrashLoggerMac, LLCrashLoggerLinux. Of course it's completely up to the frame() method whether to yield control; none of those in fact do. Honor protocol by returning true (frame() is done), even though each one's main() caller ignores the return value. In fact LLCrashLoggerWindows::mainLoop() wasn't using the return protocol correctly anyway, returning wParam or 0 or 1 -- possibly because the return protocol was never explicitly documented. It should always return true: "I'm done, don't call me again." | |||
2016-06-29 | merge changes for buildscripts | Oz Linden | |
2016-06-29 | MAINT-2141 Sky Cloud Coverage reverts to 0.27 when toggling view back to ↵ | andreykproductengine | |
region sky | |||
2016-06-29 | SL-426 - strings.xml fix for pec attachments, SL-402 - diagnostics for ↵ | Brad Payne (Vir Linden) | |
investigation | |||
2016-06-28 | Fixed up broken XML for Web Content Browser test video URL buttons | callum_linden | |
2016-06-27 | MAINT-6518 allow negative numbered chat channels | andreykproductengine | |
2016-06-27 | SL-124 - code cleanup | Brad Payne (Vir Linden) | |
2016-06-25 | SL-344 - fixes for scaling of mEye and mAltEye bones | Brad Payne (Vir Linden) | |
2016-06-24 | SL-124 - comment cleanup, fixed a crash if skinned mesh has no valid joint ↵ | Brad Payne (Vir Linden) | |
names defined | |||
2016-06-24 | MAINT-6487 After editing an object with a large number of contents, contents ↵ | andreykproductengine | |
in any object will not load until relog. | |||
2016-06-24 | Merged in rider_linden/viewer-bear (pull request #13) | Andrey Lihatskiy | |
MAINT-6521: A compare against a static const empty string causes segfault in integration tests on Mac and Linux. | |||
2016-06-24 | Merge | Rider Linden | |
2016-06-24 | MAINT-6521: A compare against a static const empty string causes segfault in ↵ | Rider Linden | |
integration tests on Mac and Linux. Use empty() test instead. | |||
2016-06-24 | SL-344 - additional slider updates, including scaling of mEye bones | Brad Payne (Vir Linden) | |
2016-06-24 | Merged in lindenlab/viewer-bear | AndreyL ProductEngine | |
2016-06-23 | Merge | Rider Linden | |
2016-06-23 | MAINT-6521: Allow anonymous connections to bypass the dependency and order ↵ | Rider Linden | |
tracking. | |||
2016-06-23 | MAINT-6516 FIXED Crash in updateEditFieldValues() while in Pathfinding -> ↵ | Mnikolenko Productengine | |
Linksets floater | |||
2016-06-22 | SL-117, SL-344 - additional proposed updates for skeleton, sliders | Brad Payne (Vir Linden) | |
2016-06-22 | Merge | Rider Linden | |
2016-06-22 | MAINT-6385: Ensure that VMM initialization only happens once after login and ↵ | Rider Linden | |
that it will only happen after the region caps have been received. | |||
2016-06-22 | Merged in lindenlab/viewer-lynx | AndreyL ProductEngine | |
2016-06-22 | MAINT-6515 Viewer doesn't store scripted null string settext updates for ↵ | andreykproductengine | |
Metadata close refreshing. | |||
2016-06-22 | MAINT-1768 Allow sharing objects from trash folder | Mnikolenko Productengine | |
2016-06-22 | MAINT-6514 Appearance Mode Body Parts Select shows wrong type | andreykproductengine | |
2016-06-21 | SL-117 - possible skeleton changes for endpoints, jaw, eyelids | Brad Payne (Vir Linden) | |
2016-06-21 | SL-353 - anim_tool.py has more reasonable default location for skeleton and ↵ | Brad Payne (Vir Linden) | |
lad files | |||
2016-06-21 | MAINT-6513 Math rounding errors | andreykproductengine | |
2016-06-17 | remove deliberate syntax error | Oz Linden | |
2016-06-17 | deliberate syntax error | Oz Linden | |
2016-06-17 | simplify early failures | Oz Linden | |
2016-06-17 | SL-398 - fix for eyelash-related sliders in avatar_lad.xml | Brad Payne (Vir Linden) | |
2016-06-17 | introduce deliberate error for buildscript testing | Oz Linden | |
2016-06-17 | MAINT-6490 Newly created inventory reports version mismatch | andreykproductengine | |
2016-06-21 | MAINT-6497 FIXED land buy disqualification missing warning icon | Mnikolenko Productengine | |
2016-06-17 | MAINT-6509 Viewer still allows you to create a group with leading spaces in ↵ | Mnikolenko Productengine | |
the group name | |||
2016-06-16 | remove experimental redirects | Oz Linden | |
2016-06-16 | MAINT-6511 Added a null check in LLFloaterView::restoreAll | AndreyL ProductEngine | |
2016-06-16 | MAINT-85 Using mouse scroll while renaming item moves it out of My Inventory tab | andreykproductengine | |
2016-06-16 | MAINT-6495 Problem with saving notecard embedded in object | Mnikolenko Productengine | |
2016-06-16 | Merged in lindenlab/viewer-lynx | AndreyL ProductEngine | |
2016-06-15 | MAINT-6486: Use literal string. Don't try and look it up. | Rider Linden | |
2016-06-15 | MAINT-6380, SL-399 - vertical flicker related to joint offsets for mPelvis. ↵ | Brad Payne (Vir Linden) | |
Having a joint offset for mPelvis is still a bad idea, but it won't cause as much flickering now. | |||
2016-06-15 | MAINT-6460 additional header check | andreykproductengine | |
2016-06-14 | SL-353 - additional anim_tool.py option, --num_pos, to enable creation of ↵ | Brad Payne (Vir Linden) | |
large test files | |||
2016-06-14 | MAINT-5040 Warning "ll_apr_warn_status: APR: Connection refused" continually ↵ | Mnikolenko Productengine | |
appears on Linux | |||
2016-06-13 | Merged in rider_linden/viewer-lynx (pull request #26) | Andrey Lihatskiy | |
2016-06-13 | Merge MAINT-6486 | Rider Linden | |
2016-06-13 | MAINT-6366 - diagnostics related to animation asset downloads | Brad Payne (Vir Linden) | |
2016-06-10 | Make string ref in HandleScriptUserData const | Rider Linden | |