Uploaded image for project: 'Snowstorm'
  1. Snowstorm
  2. STORM-1831 Obtain LSL syntax table from simulator so that it is always up to date
  3. STORM-2000

LSL arguments presented by LSLSyntax viewer's tooltips incorrectly appear in alphabetical order

    XMLWordPrintableJSON

    Details

    • Type: Story Defect
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: Active
    • Labels:
      None
    • Environment:

      Description

      While auditing the syntax highlighting of every LSL keyword in this viewer, I noticed that some functions have the arguments presented in the incorrect order. For example, llCastRay() should show this:

      list llCastRay( vector Start, vector End, list Options );

      but instead I see this in the function tooltip:

      list llCastRay( vector End, list Options, vector Start );

      It appears that the arguments are being presented in alphabetical order of the variable name instead of the actual order.

      I noticed a few other functions have the same issue:

      • llClearLinkMedia() arguments reversed: should be "integer llClearLinkMedia( integer link, integer face );"
      • llAxisAngle2Rot arguments reversed: should be "rotation llAxisAngle2Rot( vector axis, float angle );"
        There are probably several other cases, but I stopped looking after noticing this alphabetical sorting trend.

      When I query the LSLSyntax capability manually, I see the correct order for these functions under the 'arguments' map. However, the cached viewer syntax file shows the incorrect order.

      I believe that the viewer is sorting the arguments map alphabetically to cause this bug, in addition to sorting functions by name. I think this sorting is unnecessary and shouldn't be done at any scope within the xml.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                maestro.linden Maestro Linden
                Contributor:
                Ima Mechanique
              • Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: