From: "Pierre Muller" <muller@ics.u-strasbg.fr>
To: "'Eli Zaretskii'" <eliz@gnu.org>
Cc: <gdb-patches@sourceware.org>
Subject: RE: [RFA] Handle BINOP_INTDIV in valarith.c
Date: Mon, 21 Jan 2008 10:45:00 -0000 [thread overview]
Message-ID: <002401c85c1a$b1997b30$14cc7190$@u-strasbg.fr> (raw)
In-Reply-To: <uir1tpox2.fsf@gnu.org>
> If this patch is approved, please document this operator in the
> "Pascal" node of the user manual.
Hi Eli,
as I got the approval of Joel, I
committed this patch last week.
But I had an email connection problem
last week that prevented me from replying to
your email.
Concerning the pascal 'div' operator
subject of my patch, I must admit that
there is no reason to add it to 'pascal' node,
because there is basically nothing in that node yet,
and the little that is present is obsolete anyway.
I would need to rewrite completely, I will try to work on this.
The BINOP_INTDIV seems to be used also
in fortran language (f-lang.c:237)
and modula-2 language (m2-exp.y:430 and m2-lang.c:290)
But there seems to no test this feature, as I saw no
testsuite change.
I will add a pascal testsuite that
does test this feature.
I have another problem with the '/' operator:
for pascal and contrary to C language
'/' is always returning a float value, even if both
left and right nodes are integers.
I will send a separate patch for
a new binop called BINOP_FLOATDIV
that will force the result of 'a / b' to be a
float in all cases.
This is the right thing to do for pascal,
but I don't know about the other languages:
do fortran, Ada, Modula-2 or java
allow 'a / b' for a or b of integer types?
Pierre Muller
Pascal language maintainer.
next prev parent reply other threads:[~2008-01-21 10:45 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-01-16 14:14 Pierre Muller
2008-01-16 19:11 ` Eli Zaretskii
2008-01-21 10:45 ` Pierre Muller [this message]
2008-01-23 18:25 ` Joel Brobecker
2008-01-23 22:36 ` Pierre Muller
2008-01-23 23:09 ` Joel Brobecker
2008-01-23 23:55 ` Pierre Muller
2008-01-24 1:30 ` Joel Brobecker
2008-01-23 19:07 ` Tom Tromey
2008-01-23 23:00 ` Pierre Muller
2008-01-24 0:27 ` Tom Tromey
2008-01-17 11:58 ` Joel Brobecker
2008-01-17 12:04 ` Joel Brobecker
2008-01-18 16:27 ` Eli Zaretskii
2008-01-21 15:04 ` Pierre Muller
2008-01-25 13:07 ` [RFA] Handle BINOP_INTDIV in eval.c Pierre Muller
2008-01-30 1:01 ` Daniel Jacobowitz
2008-01-30 7:35 ` Pierre Muller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='002401c85c1a$b1997b30$14cc7190$@u-strasbg.fr' \
--to=muller@ics.u-strasbg.fr \
--cc=eliz@gnu.org \
--cc=gdb-patches@sourceware.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox