| Age | Commit message (Collapse) | Author | |
|---|---|---|---|
| 2011-10-14 | Backed out changeset b782a75c99e6 | Dave Parks | |
| 2011-02-05 | Introduces a LLThreadLocalData class that can be | Aleric Inglewood | |
| accessed through the static LLThread::tldata(). Currently this object contains two (public) thread-local objects: a LLAPRRootPool and a LLVolatileAPRPool. The first is the general memory pool used by this thread (and this thread alone), while the second is intended for short lived memory allocations (needed for APR). The advantages of not mixing those two is that the latter is used most frequently, and as a result of it's nature can be destroyed and reconstructed on a "regular" basis. This patch adds LLAPRPool (completely replacing the old one), which is a wrapper around apr_pool_t* and has complete thread-safity checking. Whenever an apr call requires memory for some resource, a memory pool in the form of an LLAPRPool object can be created with the same life-time as this resource; assuring clean up of the memory no sooner, but also not much later than the life-time of the resource that needs the memory. Many, many function calls and constructors had the pool parameter simply removed (it is no longer the concern of the developer, if you don't write code that actually does an libapr call then you are no longer bothered with memory pools at all). However, I kept the notion of short-lived and long-lived allocations alive (see my remark in the jira here: https://jira.secondlife.com/browse/STORM-864?focusedCommentId=235356&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-235356 which requires that the LLAPRFile API needs to allow the user to specify how long they think a file will stay open. By choosing 'short_lived' as default for the constructor that immediately opens a file, the number of instances where this needs to be specified is drastically reduced however (obviously, any automatic LLAPRFile is short lived). *** Addressed Boroondas remarks in https://codereview.secondlife.com/r/99/ regarding (doxygen) comments. This patch effectively only changes comments. Includes some 'merge' stuff that ended up in llvocache.cpp (while starting as a bug fix, now only resulting in a cleanup). *** Added comment 'The use of apr_pool_t is OK here'. Added this comment on every line where apr_pool_t is correctly being used. This should make it easier to spot (future) errors where someone started to use apr_pool_t; you can just grep all sources for 'apr_pool_t' and immediately see where it's being used while LLAPRPool should have been used. Note that merging this patch is very easy: If there are no other uses of apr_pool_t in the code (one grep) and it compiles, then it will work. *** Second Merge (needed to remove 'delete mCreationMutex' from LLImageDecodeThread::~LLImageDecodeThread). *** Added back #include <apr_pools.h>. Apparently that is needed on libapr version 1.2.8., the version used by Linden Lab, for calls to apr_queue_*. This is a bug in libapr (we also include <apr_queue.h>, that is fixed in (at least) 1.3.7. Note that 1.2.8 is VERY old. Even 1.3.x is old. *** License fixes (GPL -> LGPL). And typo in comments. Addresses merov's comments on the review board. *** Added Merov's compile fixes for windows. | |||
| 2011-02-01 | merge changes back from beta branch | Oz Linden | |
| 2011-01-24 | a less brain dead fix for CHOP-369 | Andrew A. de Laix | |
| 2011-01-24 | fix CHOP-369: catch case of synchronous download failure. | Andrew A. de Laix | |
| 2011-01-20 | merge changes from viewer-beta | Oz Linden | |
| 2011-01-14 | fix CHOP-366, on temporary errors (e.g. version manager returns other than ↵ | Andrew A. de Laix | |
| 200) show error dialog instructing the user to manually install the latest viewer. | |||
| 2011-01-13 | STORM-477 : backout changeset 6f5cb303d3e2 | Merov Linden | |
| 2011-01-11 | STORM-477 FIXED Re-implemented LLDir::getNextFileInDir() as an iterator object. | Seth ProductEngine | |
| - Replaced all existing usages of LLDir::getNextFileInDir() with the new directory iterator object. - Removed platform specific LLDir::getNextFileInDir() implementation. | |||
| 2010-12-20 | fix windows build? | Andrew A. de Laix | |
| 2010-12-17 | Better fix for CHOP-286 - reenabled bandwidth limits on linux now that we've ↵ | brad kittenbrink | |
| fixed the freeze. | |||
| 2010-12-17 | Temporary workaround for CHOP-286: bandwidth limits freeze the downloader ↵ | brad kittenbrink | |
| thread on linux | |||
| 2011-01-06 | STORM-826 (workaround): correct mixed and dos-style line endings | Oz Linden | |
| 2010-12-17 | Automated merge with https://bitbucket.org/mani_linden/viewer-development | Andrew A. de Laix | |
| 2010-12-17 | ich bin stupid; this should actually fix the windows build. | Andrew A. de Laix | |
| 2010-12-17 | don't rely on stream cast to bool behavior; use explicit fail call. | Andrew A. de Laix | |
| 2010-12-17 | and don't break other builds. | Andrew A. de Laix | |
| 2010-12-17 | fix windows build (suppress warning from lexical_cast) | Andrew A. de Laix | |
| 2010-12-16 | Vary install failed message depending on whether it was required or not. | Andrew A. de Laix | |
| 2010-12-14 | Merge | brad kittenbrink | |
| 2010-12-14 | Record update version in download marker so it can be recalled if resumed in ↵ | Andrew A. de Laix | |
| another viewer session. | |||
| 2010-12-10 | Defensive coding for linux updater script for consistency with alain's work ↵ | brad kittenbrink | |
| on the mac script. Should be safer if the user is installing to a path with spaces in it. | |||
| 2010-12-10 | fix quoting in script to work with spaces in directory names. | Andrew A. de Laix | |
| 2010-12-10 | fix working directory in install script and remove dependency on open option ↵ | Andrew A. de Laix | |
| --args which is 10.6 only. Also fix erroneous check in process launcher which was mistakenly reporting a failed execution of the new updater script. | |||
| 2010-12-10 | no bandwidth limit for required downloads. | Andrew A. de Laix | |
| 2010-12-10 | ui improvements to more closely match UX design. | Andrew A. de Laix | |
| 2010-12-07 | limit dowload bandwidth to 'Maximum bandwidth' setting | Andrew A. de Laix | |
| 2010-12-07 | push required flag into download data for later use. | Andrew A. de Laix | |
| 2010-12-03 | Fix for coding standard violations and build error on windows. | brad kittenbrink | |
| 2010-12-02 | expose update available method. | Andrew A. de Laix | |
| 2010-12-01 | state change events for updater service. | Andrew A. de Laix | |
| 2010-12-01 | download progress events. | Andrew A. de Laix | |
| 2010-11-29 | Attempted fix for CHOP-223: fixed executable permissions on update_install ↵ | brad kittenbrink | |
| script for mac and linux. | |||
| 2010-11-19 | Merged latest lindenlab/viewer-development with ↵ | brad kittenbrink | |
| mani_linden/viewer-development for bacground updater work. | |||
| 2010-11-18 | CHOP-135 Bug fixes. | Mark Palange (Mani) | |
| 2010-11-18 | merge | Mark Palange (Mani) | |
| 2010-11-18 | Fixing bugs discovered in merge with viewer development | Mark Palange (Mani) | |
| 2010-11-18 | conform to coding standard. | Andrew A. de Laix | |
| 2010-11-18 | Merge with viewer-development | Mark Palange (Mani) | |
| 2010-11-18 | inform user on failed install--needs proper user dialog ;-) | Andrew A. de Laix | |
| 2010-11-18 | Dummy out LLUpdateDownloader::cancel() too for testing. | Nat Goodspeed | |
| 2010-11-17 | Work on CHOP-135: Hooking up setting UpdaterServiceActive to functionality. ↵ | brad kittenbrink | |
| Paired with Mani. Toggling the setting now calls LLUpdaterService::startChecking() and LLUpdaterService::stopChecking(), which enable and disable the service. | |||
| 2010-11-17 | post events for dowload success and error. | Andrew A. de Laix | |
| 2010-11-17 | CHOP-203 Deleting the update file after installer run. | Mark Palange (Mani) | |
| 2010-11-16 | CHOP-209 Added marker creation, fixed updater crash bug | Mark Palange (Mani) | |
| 2010-11-16 | Automated merge with http://bitbucket.org/mani_linden/viewer-development | Mark Palange (Mani) | |
| 2010-11-16 | remove downloaded file on error. | Andrew A. de Laix | |
| 2010-11-16 | merge | Mark Palange (Mani) | |
| 2010-11-16 | CHOP-179 Linux install uses linux-updater. Rev. by Alain | Mark Palange (Mani) | |
| 2010-11-16 | clean up installer file after install. | Andrew A. de Laix | |
