Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: Phoenix Firestorm 4.2.2, Phoenix Firestorm 4.3.0
    • Fix Version/s: Phoenix Firestorm 4.7.0
    • Component/s: None
    • Labels:
    • SL Avatar Name:
      PixelProphet Lane
    • Reported In:
      Firestorm 4.2.2.29837 Pathfinding Hotfix

      Description

      This issue is being reported for Darling Brody.

      It seems that the LSL PreProcessor will parse a string containing 2 forward slashes // as a comment, therefore cutting the closing double-quote off, and producing a syntax error. Apparently using concatenation "http://bla" + "bla bla" does not break the code.

      Below is an example LSL Script to reproduce the problem:

      //start_unprocessed_text
      /*default
      {
          state_entry()
          {
              
              string error = "everything after the double slash will become a comment /|/ even though it's inside a string.
              this is a big problem when typing a url http:/|/www.whatever.com if you break it over a line for readability
              which I obviourly did.";
              llSay(0,error);
          }
      }
      */
      //end_unprocessed_text
      //nfo_preprocessor_version 0
      //program_version Phoenix Firestorm-Release v4.2.2.29837 - Darling Brody
      //mono
      
      
       
      default
      {
          state_entry()
          {
              
              string error = "everything after the double slash will become a comment 
              this is a big problem when typing a url http:
              which I obviourly did.";
              llSay(0,error);
          }
      }
      

        Issue Links

          Activity

          Hide
          jessica_lyon Jessica Lyon added a comment -

          Assigning to Chalice in hopes she'll fix it in NaCl and send the fix back to us. We don't have the expertise to fix it atm.

          Show
          jessica_lyon Jessica Lyon added a comment - Assigning to Chalice in hopes she'll fix it in NaCl and send the fix back to us. We don't have the expertise to fix it atm.
          Hide
          ansariel.hiller Ansariel Hiller added a comment -

          This should be fixed with the fix for FIRE-15681 in rev. 45102 (http://hg.phoenixviewer.com/phoenix-firestorm-lgpl/rev/00a3246f290c).

          Show
          ansariel.hiller Ansariel Hiller added a comment - This should be fixed with the fix for FIRE-15681 in rev. 45102 ( http://hg.phoenixviewer.com/phoenix-firestorm-lgpl/rev/00a3246f290c ).

            People

            • Assignee:
              ansariel.hiller Ansariel Hiller
              Reporter:
              pixelprophet PixelProphet
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: