Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | Use GetInventoryItem() in llRezAtRoot rather than iterating through a cloned ↵ | Justin Clark-Casey (justincc) | 2012-07-04 | 1 | -40/+36 | |
| | | | | dictionary | |||||
* | refactor: In llGetNotecardLine() use existing GetInventoryItem() rather than ↵ | Justin Clark-Casey (justincc) | 2012-07-04 | 1 | -10/+4 | |
| | | | | inspecting a clone of the TaskInventory dictionary that was not cloned thread-safe | |||||
* | refactor: In llGetNumberOfNotecardLines() use existing GetInventoryItem() ↵ | Justin Clark-Casey (justincc) | 2012-07-04 | 1 | -10/+4 | |
| | | | | rather than inspecting a clone of the TaskInventory dictionary that was not cloned thread-safe | |||||
* | refactor: use existing GetInventoryItem() in GetScriptByName(), itself ↵ | Justin Clark-Casey (justincc) | 2012-07-04 | 1 | -13/+9 | |
| | | | | renamed from ScriptByName() | |||||
* | refactor: In llGetInventoryType() use existing GetInventoryItem() | Justin Clark-Casey (justincc) | 2012-07-04 | 1 | -11/+5 | |
| | ||||||
* | refactor: In llGetInventoryCreator() use existing GetInventoryItem() | Justin Clark-Casey (justincc) | 2012-07-04 | 1 | -12/+8 | |
| | ||||||
* | refactor: In llGetInventoryPermMask() use existing GetInventoryItem() | Justin Clark-Casey (justincc) | 2012-07-04 | 1 | -24/+21 | |
| | ||||||
* | refactor: in llGetInventoryPermMask use existing GetInventoryItem() | Justin Clark-Casey (justincc) | 2012-07-04 | 1 | -23/+19 | |
| | ||||||
* | refactor: In llRemoteLoadScriptPin() use existing GetInventoryItem() | Justin Clark-Casey (justincc) | 2012-07-04 | 1 | -20/+5 | |
| | ||||||
* | refactor: replace use of LSL_Api.GetTaskInventoryItem() with existing ↵ | Justin Clark-Casey (justincc) | 2012-07-04 | 1 | -22/+11 | |
| | | | | GetInventoryItem() | |||||
* | refactor: In llGetInventoryKey() use existing GetInventoryItem() | Justin Clark-Casey (justincc) | 2012-07-04 | 1 | -15/+9 | |
| | ||||||
* | In llMessageLinked() use GetInventoryItems() rather than cloning ↵ | Justin Clark-Casey (justincc) | 2012-07-04 | 1 | -4/+2 | |
| | | | | | | TaskInventory directory GetInventoryItems() returns a new list and so is equivalent, and creates this list under lock whereas Clone() is not thread-safe | |||||
* | In llRequestInventoryData() use GetInventoryItems() rather than cloning ↵ | Justin Clark-Casey (justincc) | 2012-07-04 | 1 | -3/+2 | |
| | | | | | | TaskInventory directory GetInventoryItems() returns a new list and so is equivalent, and creates this list under lock whereas Clone() is not thread-safe | |||||
* | refactor: In llCollisionSound() use existing GetInventoryItem() method ↵ | Justin Clark-Casey (justincc) | 2012-07-04 | 1 | -11/+5 | |
| | | | | rather than have it iterate through TaskInventory itself. | |||||
* | refactor: In llRemoveInventory() use existing GetInventoryItem() method ↵ | Justin Clark-Casey (justincc) | 2012-07-04 | 1 | -14/+9 | |
| | | | | rather than have it iterate through TaskInventory itself. | |||||
* | refactor: make llGiveInventory() use existing GetInventoryItem() method ↵ | Justin Clark-Casey (justincc) | 2012-07-04 | 1 | -24/+11 | |
| | | | | rather than iterate through TaskInventory itself. | |||||
* | refactor: rename Watchdog.WATCHDOG_TIMEOUT_MS to DEFAULT_WATCHDOG_TIMEOUT_MS ↵ | Justin Clark-Casey (justincc) | 2012-07-04 | 1 | -2/+2 | |
| | | | | to reflect what it actually is | |||||
* | Correct spelling mistake m_BanedViewers to m_BannedViewers | Justin Clark-Casey (justincc) | 2012-07-03 | 1 | -4/+4 | |
| | ||||||
* | minor: Add some method doc to HasGroupChanged and Schedule ↵ | Justin Clark-Casey (justincc) | 2012-07-03 | 1 | -2/+21 | |
| | | | | GroupForFull/PartUpdate() to indicate when region modules need to invoke them | |||||
* | Fix issue in database tests where sogs being stored are not in a scene. | Justin Clark-Casey (justincc) | 2012-07-03 | 1 | -1/+1 | |
| | | | | This puts an extra m_part.ParentGroup.Scene == null check at the top of SceneObjectPartInventory.QueryScriptStates() | |||||
* | Adds a list of viewers that are allowed or banned from the region. | Michelle Argus | 2012-07-02 | 1 | -0/+65 | |
| | | | | Signed-off-by: Melanie <melanie@t-data.com> | |||||
* | Add preservation of running state of scripts when drag-copying. | Melanie | 2012-07-01 | 3 | -2/+50 | |
| | ||||||
* | Remove some mono compiler warnings | Justin Clark-Casey (justincc) | 2012-06-30 | 1 | -2/+3 | |
| | ||||||
* | Removing unused handling of incoming create object by userID and itemID only. | Justin Clark-Casey (justincc) | 2012-06-30 | 3 | -48/+1 | |
| | | | | | It appears this was never actually used since attachments were rezzed in other code. This was never available on remote simulator comms, only local. | |||||
* | Remove code listed for removal in 0.7.3 that handled script restart for ↵ | Justin Clark-Casey (justincc) | 2012-06-30 | 1 | -16/+0 | |
| | | | | incoming attachments from pre-fatpack regions (versions of OpenSimulator more than a year old) | |||||
* | Move update of the final optional ODE total frame stat inside the OdeLock ↵ | Justin Clark-Casey (justincc) | 2012-06-29 | 1 | -3/+3 | |
| | | | | rather than outside to avoid a very occasional race condition with the stat collection thread | |||||
* | refactor: rename _collisionEventPrim to m_collisionEventActors and ↵ | Justin Clark-Casey (justincc) | 2012-06-29 | 1 | -12/+12 | |
| | | | | | | _collisionEventPrimChanges to m_collisionEventActorsChanges to reflect their actual contents. These dictionaries handle all actor types, not just physical prims. | |||||
* | Fix a regression where we stopped removing avatars from collision event ↵ | Justin Clark-Casey (justincc) | 2012-06-29 | 1 | -1/+1 | |
| | | | | | | | reporting on logout, rather than stopping clearing their collision events. This occurred in b18c8c8 (Thu May 17 2012). This was a cause of very occasional race conditions and likely memory leakage as clients came and went from the region. | |||||
* | Add IScene.Name for code clarity to replace the RegionInfo.RegionName used ↵ | Justin Clark-Casey (justincc) | 2012-06-29 | 3 | -32/+33 | |
| | | | | in many, many log messages. | |||||
* | Change AttachmentsModule.DetachSingleAttachmentToInv() to accept a SOG ↵ | Justin Clark-Casey (justincc) | 2012-06-28 | 5 | -52/+58 | |
| | | | | | | | directly instead of an item ID to then shuffle through attachments, saving CPU busywork. Almost all callers already had the sog to hand. Still checking that it's really an attachment, but now by inspecting SOG.AttachedAvatar | |||||
* | Rather than iterating through all SOGs in the scene looking for the one that ↵ | Justin Clark-Casey (justincc) | 2012-06-28 | 1 | -24/+17 | |
| | | | | matches out fromItemID on detach, go through the agent's attachment sog list instead. | |||||
* | Remove code that tried to delete an attachment back to inventory if ↵ | Justin Clark-Casey (justincc) | 2012-06-28 | 1 | -6/+1 | |
| | | | | | | | RezSingleAttachmentFromInventoryInternal() returned null. null would only ever be returned if the item couldn't be located within inventory and this would happen immediately. In this case, derezzing wouldn't work anyway since there is no item to derez. | |||||
* | If rest of first line after colon is blank then still warn about running in ↵ | Justin Clark-Casey (justincc) | 2012-06-28 | 1 | -11/+18 | |
| | | | | | | | XEngine if engine specified does not exist. This is to take account of situations where the user was intending to specify a script engine using colon using its default language. This probably generates few false positive as scripts are less likely to end a first line colon with a comment for other purposes. | |||||
* | Avoid reporting false positives when a colon is in a comment in the first ↵ | Justin Clark-Casey (justincc) | 2012-06-28 | 1 | -17/+41 | |
| | | | | | | | | | line of a script where the user was not trying to select a different script engine. This works by only posting the "Selected engine unavailable" message if we're falling back on XEngine and the language is one handled by XEngine. In cases where the language is not handled or not allowed, the user will still be notified by the later compiler error. This avoids the overwhelming majority of false positives where the first line contains a : for other reasons (e.g. source control systems, vim settings, etc.) Ultimately, I think it would be better to detect script language/engine with a mechanism that didn't just rely on : detection (e.g like #! in unix scripts). | |||||
* | minor: reuse colon index calculation in XEngine.OnRezScript. The index if a ↵ | Justin Clark-Casey (justincc) | 2012-06-28 | 1 | -2/+2 | |
| | | | | colon is found on the first line will always be the same as for the whole script. | |||||
* | Fix issue in InventoryArchiveTestCase where it didn't call down to ↵ | Justin Clark-Casey (justincc) | 2012-06-27 | 1 | -1/+2 | |
| | | | | OpenSimTestCase.SetUp() | |||||
* | refactor: Move ScenePresence <-> AgentData attachments copying code into ↵ | Justin Clark-Casey (justincc) | 2012-06-27 | 4 | -37/+74 | |
| | | | | AttachmentsModule. | |||||
* | Automatically disable log4net before each regression test so that logging is ↵ | Justin Clark-Casey (justincc) | 2012-06-27 | 2 | -4/+2 | |
| | | | | | | | confined to a single test if it's turned on. This involves making test classes inherit from a common OpenSimTestCase. This will be applied to more classes as required. | |||||
* | Replace "kill uuid" console command with the more consistent "delete object ↵ | Justin Clark-Casey (justincc) | 2012-06-26 | 1 | -56/+0 | |
| | | | | uuid", which was present in the last opensim release. | |||||
* | Fix output for help on some object region console commands | Justin Clark-Casey (justincc) | 2012-06-26 | 1 | -3/+6 | |
| | ||||||
* | refactor: Remove unnecessary AttachmentModuleTests.m_userId in favour of ↵ | Justin Clark-Casey (justincc) | 2012-06-26 | 1 | -40/+26 | |
| | | | | local variables | |||||
* | refactor: Use local attachment module variables instead of global m_attMod. | Justin Clark-Casey (justincc) | 2012-06-26 | 1 | -11/+10 | |
| | | | | This also avoids confusion between tests where one sets up m_attMod and another accidentally uses it after failing to set one up itself. | |||||
* | refactor: make m_presence a local variable in all AttachmentsModuleTests ↵ | Justin Clark-Casey (justincc) | 2012-06-26 | 1 | -26/+25 | |
| | | | | since it doesn't need to be global and some tests set up more than one sp | |||||
* | Add regression test to check that attachments in source region are deleting ↵ | Justin Clark-Casey (justincc) | 2012-06-26 | 2 | -18/+120 | |
| | | | | when an agent teleports to a neighbouring region | |||||
* | Merge branch 'master' of ssh://opensimulator.org/var/git/opensim | Justin Clark-Casey (justincc) | 2012-06-26 | 1 | -0/+5 | |
|\ | ||||||
| * | Add stub for llCastRay | BlueWall | 2012-06-26 | 1 | -0/+5 | |
| | | ||||||
* | | If crossing attachments into another region pre-fatpack, clone objects ↵ | Justin Clark-Casey (justincc) | 2012-06-26 | 1 | -17/+26 | |
|/ | | | | | | | before changing properties to avoid hud display race condition with update threads. This matches behaviour in fatpack crossing, where attachments are cloned before their properties are changed. This only applies to crossings to simulators running code released before April 2011. | |||||
* | minor: update currently commented out log message at top of ↵ | Justin Clark-Casey (justincc) | 2012-06-25 | 1 | -1/+3 | |
| | | | | AvatarFactoryModule.SetAppearance() for future use | |||||
* | When attachments are being saved and deleted for a closing root agent, ↵ | Justin Clark-Casey (justincc) | 2012-06-25 | 4 | -29/+42 | |
| | | | | | | delete first to avoid a hud race condition with update threads. If delete doesn't occur first then the update thread can outrace the IsAttachment = false necessary to save attachments and send hud artifacts to other viewers. | |||||
* | In AttachmentsModule.DetachSingleAttachmentToInvInternal(), remove ↵ | Justin Clark-Casey (justincc) | 2012-06-25 | 3 | -2/+42 | |
| | | | | | | | | | attachment before changing properties for correct inventory serialization. Serialization of attachments requires IsAttachment = false so that correct positions are serialized instead of avatar position. However, doing this when a hud is still attached allows race conditions with update threads, resulting in hud artifacts on other viewers. This change sets SOG.IsDeleted before serialization changes take place (IsDeleted itself is not a serialized property). LLClientView then screens out any deleted SOGs before sending updates to viewers. |