#83 Macro expansion not reliable in tools configuration

None
pending
orwelldevcpp
None
2013-06-12
2013-05-26
CommonTater
No

Version: 5.4.2 (and previous)
OS version: Win7 pro x64 sp1
Error reported: none
Stack trace: none
Steps needed to reproduce the problem:

When setting up tools the <WORDXY> macro is not always expanded.

For example... in Tools.ini

Title=WinAPI Documentation
Program=C:\Program Files (x86)\Common Files\microsoft shared\Help 9\dexplore.exe
WorkDir=C:\Program Files (x86)\Common Files\microsoft shared\Help 9
Params=/helpcol MS-Help://MS.W7SDK.1033 /launchfkeywordtopic <WORDXY>

Will result in Dexplore.exe looking for the keyword <WORDXY> rather than the word under the edit cursor.

Also.. a suggestion... allow assigning hot keys to tools... The above just begs to be assigned to F1 ...

Discussion

  • CommonTater
    CommonTater
    2013-05-26

    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -4,7 +4,7 @@
     Stack trace:  none
     Steps needed to reproduce the problem:
    
    -When setting up tools the <WORDXY> macro is not always expanded. 
    +When setting up tools the <WORDXY\> macro is not always expanded. 
    
     For example...  in Tools.ini
    
    • Milestone: -->
     
  • orwelldevcpp
    orwelldevcpp
    2013-05-27

    TODO:
    - Convert the example command line to a read only edit field. Easy.
    - Allow users to use custom shortcuts. Easy.
    - Fix this bug. Hard.

     
  • orwelldevcpp
    orwelldevcpp
    2013-05-27

    • status: open --> pending
    • assigned_to: orwelldevcpp
     
  • CommonTater
    CommonTater
    2013-05-27

    If you notice the second parameter for Dexplore is a url of sorts. What crossed my mind was that it gave up parsing at that point and just treated everything after as part of the url.

    Other macros such as the current filename, were correctly expanded in that spot.

    If it will help, the documentation for Dexplore is here...
    http://msdn.microsoft.com/en-us/library/bb165687(v=vs.80).aspx

     
    Last edit: CommonTater 2013-05-27
  • orwelldevcpp
    orwelldevcpp
    2013-05-27

    Might be possible, but it's simply doing a text based replace.

    When I get home, I'll review the parsing code and send debugging versions wich you can try.

     
  • CommonTater
    CommonTater
    2013-05-27

    Sounds good ... thanks.

     
  • orwelldevcpp
    orwelldevcpp
    2013-06-12

    Here's a preview version of 5.4.3 that allows you to see what is happening in that edit box.

    It's not much after two weeks, but as usual, my free time is a limit to zero.