man, 2004-11-22 kl. 17:34 skrev Morten:
> man, 2004-11-22 kl. 17:30 skrev Javier S. Lopez:
> > Morten / Datagruppen MultiMED <[EMAIL PROTECTED]> writes:
> > 
> > > man, 2004-11-22 kl. 13:31 skrev Morten:
> > >> man, 2004-11-22 kl. 13:24 skrev Paul Kinnucan:
> > >> > It's possible there is a regression. I will take a look
> > >> > this evening to see if I can reproduce the problem. 
> > >> > 
> > >> > Just to be sure, which debugger are you using? The JDEE's
> > >> > interface to jdb or JDEbug?
> > > <CUT>
> > >> I'm using the jdb.
> > >
> > >
> > > I noticed that in the code, it checks for "," .. and replaces it with
> > > "".
> > This will definetely cause the problem. I can change the code to look for 
> > ".",
> > but I wonder if there are any other characters or a better way of doing 
> > this.
> > 
> 
> Maybe "[^:digit:]" instead of "," .. but it also has to match lines like
> "9,846" .. so "," needs to be exchanged for something in the code that
> finds the lines also, I guess.

Apparently I managed to fix the error for myself with the help of the
good people in #emacs on irc.gnu.org

I havn't tested it well yet, and I havn't done a cvs diff yet.

The trick is to replace , with "[:digit:] in two places:

In jde-db-cmd-notify-response:

Change to this:
    (if (string-match 
         (concat "^  \\["
                 (oref debuggee :stack-depth)
        "\\] .*(\\([^\$\n]*\\).*:\\([0-9]*[^[:digit:]]?[0-9]+\\))")
         output)

and change jde-jdb-string-to-int(numer) like this:

(defun jde-jdb-string-to-int(number)
  "Some numbers show like, 1,200, this method remove the comma to allow
the
number to show as 1200 insteand 1"
  (if (string-match "[^[:digit:]]" number)
      (setq number (replace-match "" nil nil number)))
  (string-to-int number))

I *think* thats it!

Morten .


Reply via email to