Announcement

Collapse
No announcement yet.

Possible bug in command execution ? (ExecTBL.ini)

Collapse
X
  • Filter
  • Time
  • Show
Clear All
new posts

  • Possible bug in command execution ? (ExecTBL.ini)

    This is some code in my ExecTBL.ini
    Code:
    /, SETTINGS SCREEN
    "SET_UPD_MUSIC","BYVAR;SYGIC_PAUSEWINAMP;SETVAR;SET_MUSIC;$OPTIONS_21$<<SETVAR;SET_MUSIC;$OPTIONS_22$"
    "SET_UPD_SWITCH","BYVAR;SYGIC_SWITCHSCREEN;SETVAR;SET_SWITCH;$OPTIONS_31$<<SETVAR;SET_SWITCH;$OPTIONS_32$"
    "SET_UPD_DELAY","EVAL;SET_TMP;strcomp("$SYGIC_SWITCHBACKTIME$";"0")||BYVAR;SET_TMP;SETVAR;SET_DELAY;$OPTIONS_411$$SYGIC_SWITCHBACKTIME$$OPTIONS_412$<<SETVAR;SET_DELAY;$OPTIONS_42$"
    "SET_UPD_OSK","BYVAR;HOSK_DEFAULT;SETVAR;SET_HOSK;$OPTIONS_61$<<SETVAR;SET_HOSK;$OPTIONS_62$"
    "SET_UPD_MEDIABASE","BYVAR;MEDIABASE_REFRESHMODE;SETVAR;SET_MEDIABASE;$OPTIONS_71$<<SETVAR;SET_MEDIABASE;$OPTIONS_72$<<SETVAR;SET_MEDIABASE;$OPTIONS_73$"
    "SET_SAVE","SAVETOSKIN;SYGIC_PAUSEWINAMP;$SYGIC_PAUSEWINAMP$||SAVETOSKIN;SYGIC_SWITCHSCREEN;$SYGIC_SWITCHSCREEN$||SAVETOSKIN;SYGIC_SWITCHBACKTIME;$SYGIC_SWITCHBACKTIME$||SAVETOSKIN;HOSK_DEFAULT;$HOSK_DEFAULT$||SAVETOSKIN;MEDIABASE_REFRESHMODE;$MEDIABASE_REFRESHMODE$"
    
    "UPDATEIND_SETUP","SET_UPD_MUSIC||SET_UPD_SWITCH||SET_UPD_DELAY||SET_UPD_OSK||SET_UPD_MEDIABASE"
    The last command throws an error. Here's part of the log when it's executed:
    PHP Code:
    08.06.2009 09:12:14Que ExecWahrmenu2.skin"UPDATEIND_SETUP||LOAD;options.skin"Falsch
    08.06.2009 09
    :12:14ExecutingSET_UPD_MUSIC
    08.06.2009 09
    :12:14Command not executedSET_UPD_MUSIC
    08.06.2009 09
    :12:14Que ExecWahrmenu2.skin"ONCOMMANDINVALID"Falsch
    08.06.2009 09
    :12:14ExecutingONCOMMANDINVALID
    08.06.2009 09
    :12:14Que ExecWahrmenu2.skin"SET_UPD_SWITCH||SET_UPD_DELAY||SET_UPD_OSK||SET_UPD_MEDIABASE"Falsch
    08.06.2009 09
    :12:14ExecutingBYVAR;SYGIC_SWITCHSCREEN;SETVAR;SET_SWITCH;Bildschirm bei Anweisungen nicht wechseln<<SETVAR;SET_SWITCH;Bei Anweisungen zum GPS Bildschirm wechseln
    08.06.2009 09
    :12:14Que ExecWahrmenu2.skin"SETVAR;SET_SWITCH;Bei Anweisungen zum GPS Bildschirm wechseln"Falsch
    08.06.2009 09
    :12:14ExecutingSETVAR;SET_SWITCH;Bei Anweisungen zum GPS Bildschirm wechseln
    08.06.2009 09
    :12:14Que ExecWahrmenu2.skin"SET_UPD_DELAY||SET_UPD_OSK||SET_UPD_MEDIABASE"Falsch
    08.06.2009 09
    :12:14Executing: EVAL;SET_TMP;strcomp("2";"0")
    08.06.2009 09:12:14Que ExecWahrmenu2.skin"BYVAR;SET_TMP;SETVAR;SET_DELAY;$OPTIONS_411$$SYGIC_SWITCHBACKTIME$$OPTIONS_412$<<SETVAR;SET_DELAY;$OPTIONS_42$"Falsch
    08.06.2009 09
    :12:14ExecutingBYVAR;SET_TMP;SETVAR;SET_DELAY;Nach Anweisungen 2 Sekunden warten<<SETVAR;SET_DELAY;Nach Anweisungen sofort zurück wechseln
    08.06.2009 09
    :12:14Que ExecWahrmenu2.skin"SETVAR;SET_DELAY;Nach Anweisungen 2 Sekunden warten"Falsch
    08.06.2009 09
    :12:14ExecutingSETVAR;SET_DELAY;Nach Anweisungen 2 Sekunden warten
    08.06.2009 09
    :12:14Que ExecWahrmenu2.skin"SET_UPD_OSK||SET_UPD_MEDIABASE"Falsch
    08.06.2009 09
    :12:14ExecutingBYVAR;HOSK_DEFAULT;SETVAR;SET_HOSK;Standard Bildschirmtastatur verwenden<<SETVAR;SET_HOSK;Kompakte Bildschirmtastatur verwenden
    08.06.2009 09
    :12:14Que ExecWahrmenu2.skin"SETVAR;SET_HOSK;Standard Bildschirmtastatur verwenden"Falsch
    08.06.2009 09
    :12:14ExecutingSETVAR;SET_HOSK;Standard Bildschirmtastatur verwenden
    08.06.2009 09
    :12:14Que ExecWahrmenu2.skin"SET_UPD_MEDIABASE"Falsch
    08.06.2009 09
    :12:14ExecutingBYVAR;MEDIABASE_REFRESHMODE;SETVAR;SET_MEDIABASE;Bei DB Refresh alle Tags importieren<<SETVAR;SET_MEDIABASE;Bei DB Refresh keine Tags speichern<<SETVAR;SET_MEDIABASE;Bei DB Refresh nur neue Tags lesen
    08.06.2009 09
    :12:14Que ExecWahrmenu2.skin"SETVAR;SET_MEDIABASE;Bei DB Refresh nur neue Tags lesen"Falsch
    08.06.2009 09
    :12:14ExecutingSETVAR;SET_MEDIABASE;Bei DB Refresh nur neue Tags lesen
    08.06.2009 09
    :12:14Que ExecWahrmenu2.skin"LOAD;options.skin"Falsch
    08.06.2009 09
    :12:14ExecutingLOAD;options.skin 
    It says that the command SET_UPD_MUSIC is invalid. If I remove the command SET_UPD_MUSIC it throws the same error for the next command (SET_UPD_SWITCH).
    I then just changed the command "UPDATEIND_SETUP" by adding an empty command to the beginning
    Code:
    "UPDATEIND_SETUP"," ||SET_UPD_MUSIC||SET_UPD_SWITCH||SET_UPD_DELAY||SET_UPD_OSK||SET_UPD_MEDIABASE"
    Then it works and doesn't throw an error:
    PHP Code:
    08.06.2009 09:15:12Que ExecWahrmenu2.skin"UPDATEIND_SETUP||LOAD;options.skin"Falsch
    08.06.2009 09
    :15:12Executing:  
    08.06.2009 09:15:12Que ExecWahrmenu2.skin"SET_UPD_MUSIC||SET_UPD_SWITCH||SET_UPD_DELAY||SET_UPD_OSK||SET_UPD_MEDIABASE"Falsch
    08.06.2009 09
    :15:12ExecutingBYVAR;SYGIC_PAUSEWINAMP;SETVAR;SET_MUSIC;Bei Anweisungen Lautstärke reduzieren<<SETVAR;SET_MUSIC;Bei Anweisungen Musik pausieren
    08.06.2009 09
    :15:12Que ExecWahrmenu2.skin"SETVAR;SET_MUSIC;Bei Anweisungen Lautstärke reduzieren"Falsch
    08.06.2009 09
    :15:12ExecutingSETVAR;SET_MUSIC;Bei Anweisungen Lautstärke reduzieren 
    Is that a bug, or am I doing something wrong ? I'm using the 06-01-2009 release. I just added this command, so I can't tell since which version this happens.

  • #2
    EXECTBL IS forward translation only...

    so... UPDATEIND_SETUP cannot translate into "SET_UPD_MUSIC"
    (technical reasons)
    in the 1st command, since its after in the list

    move the UPDATEIND_SETUP line before SET_UPD_MUSIC

    try that...

    mitch
    -Thanks
    Mitch
    www.rush2112.net

    "Did you test it in carwings??"

    Sun, Come shine my way
    May healing waters bury all my pain
    Wind, Carry me home
    The fabric of reality is tearing apart
    The piece of me that died
    Will return To live again

    Comment


    • #3
      For unseen reasons, this is sorta funny, eh mitch...hehe

      He actually made an attempt to fix this but ended up making it way worse, and decided that its not really broken b/c most skinners know that they have to make it forward translate.
      RideRunner...The #1 FE, PERIOD.

      Current Project: DFXVoice [v1.0 Released]
      Next in line: RRMedia v2.0

      DFX 5.1.1 Skin
      Appstore Link

      My RideRunner Plugins
      RRMedia
      RRExtended
      DFXVoice

      "There are no bugs - only undocumented features."

      Comment


      • #4
        Oooookeyyyy, can this be changed ? I actually think that's a really strange behaviour.

        And why does it work with the empty command ? As far as I understand your argumentation this shouldn't work either, because UPDATEIND_SETUP is still after SET_UPD_MUSIC in the list ?!

        Comment


        • #5
          I didnt make it worse, i MADE IT BETTER
          but took it out, because someoone could make a exectble that loops forever
          and hangs RR....

          because, the commands after || are seperated, requeued, and thus go though the translation loop from the top

          its always been this way... I wanted the order not to matter, but we found in cases some bad exectble coding would loop forever translating... think about it
          -Thanks
          Mitch
          www.rush2112.net

          "Did you test it in carwings??"

          Sun, Come shine my way
          May healing waters bury all my pain
          Wind, Carry me home
          The fabric of reality is tearing apart
          The piece of me that died
          Will return To live again

          Comment


          • #6
            Mitch had asked that too,as he had no idea that it worked the way it currently does.

            In an ideal world, you should be able to translate any command in that table, reguardless of where its at...
            RideRunner...The #1 FE, PERIOD.

            Current Project: DFXVoice [v1.0 Released]
            Next in line: RRMedia v2.0

            DFX 5.1.1 Skin
            Appstore Link

            My RideRunner Plugins
            RRMedia
            RRExtended
            DFXVoice

            "There are no bugs - only undocumented features."

            Comment


            • #7
              Originally posted by mitchjs View Post
              I didnt make it worse, i MADE IT BETTER
              but took it out, because someoone could make a exectble that loops forever
              and hangs RR....
              Didn't think about that. Still think it would be better if it's not just forward execution. Sadly I don't have a clever idea to avoid loops. But I'll get back to you when I have one .

              Comment


              • #8
                konrad, i did have it that order not matter, and it didnt take long for G to show me an example of what people have already in a skin, and it would cause a loop
                i do have some ideas... but for now, im gonna slide it a bit back into my memory, because it is working as it always has...
                im focusing now, on two other issues...ill revisit it im sure

                thanks for posting, because when it happend to me, thats when i learned that order mattered!!!
                -Thanks
                Mitch
                www.rush2112.net

                "Did you test it in carwings??"

                Sun, Come shine my way
                May healing waters bury all my pain
                Wind, Carry me home
                The fabric of reality is tearing apart
                The piece of me that died
                Will return To live again

                Comment


                • #9
                  Tiss... maybe if you actually used the SW outside of testing things, ya know...like in a car...
                  RideRunner...The #1 FE, PERIOD.

                  Current Project: DFXVoice [v1.0 Released]
                  Next in line: RRMedia v2.0

                  DFX 5.1.1 Skin
                  Appstore Link

                  My RideRunner Plugins
                  RRMedia
                  RRExtended
                  DFXVoice

                  "There are no bugs - only undocumented features."

                  Comment

                  Working...
                  X