topbanner_forum
  *

avatar image

Welcome, Guest. Please login or register.
Did you miss your activation email?

Login with username, password and session length
  • Thursday March 28, 2024, 2:29 pm
  • Proudly celebrating 15+ years online.
  • Donate now to become a lifetime supporting member of the site and get a non-expiring license key for all of our programs.
  • donate

Author Topic: Poss minor bug in JrDebug::ParseCommandlineArgString - version 1.01.18  (Read 12317 times)

walkerr

  • Participant
  • Joined in 2007
  • *
  • default avatar
  • Posts: 2
    • View Profile
    • Donate to Member
Been using this version, and had a problem where quoted strings were in the command line - the inquote variable never gets set false basically.

The following fix seems to work

at line 60 in my copy, I have the def of the function:

bool JrDebug::ParseCommandlineArgString(const char* commandlinestr,char *arg0,bool &foundcommandlineargs)

then from line 104:

      else if ((c==34 && inquote==true) || (inquote==false && c==32) || c==0)
         {
         //rhw-marker
         inquote=false;
         //rhw-marker

The new line surrounded by marker just to reset inquote flag once quoted arg is stored

-- Rob

mouser

  • First Author
  • Administrator
  • Joined in 2005
  • *****
  • Posts: 40,896
    • View Profile
    • Mouser's Software Zone on DonationCoder.com
    • Read more about this member.
    • Donate to Member
Great find, Rob, you're right - it's definitely needed.

Updating my version and i'll credit you in the release notes for next release :)

walkerr

  • Participant
  • Joined in 2007
  • *
  • default avatar
  • Posts: 2
    • View Profile
    • Donate to Member
You're welcome - and thanks

To be honest, I'm more of a Java programmer these days. My C++ is very rusty - JrDebug has been a big help to me untangling the rubbish code I've been writing!

- Rob