caml-list - the Caml user's mailing list
 help / color / mirror / Atom feed
From: "Nicolas Pouillard" <nicolas.pouillard@gmail.com>
To: "Aleksey Nogin" <nogin@metaprl.org>, "Caml List" <caml-list@inria.fr>
Subject: Re: 3.10.0+beta: Corrected and improved Camlp4MacroParser
Date: Tue, 17 Apr 2007 21:28:17 +0200	[thread overview]
Message-ID: <cd67f63a0704171228j15bbc76ie63db5c6c718c87c@mail.gmail.com> (raw)
In-Reply-To: <46241F07.8090602@metaprl.org>

On 4/17/07, Aleksey Nogin <nogin@metaprl.org> wrote:
> As I wrote to the list earlier, I have noticed that the
> Camlp4MacroParser in 3.10.0+beta has an unfortunate problem - when
> processing an "IFDEF" (or "IFNDEF") directive, it will execute the
> "DEFINE"s in _both_ branches, regardless of whether the test is true or
> not (this was caused by the fact that the DEFINE directives would be
> executed eagerly at parse-time).
>
> Attached is a corrected and improved version of the Camlp4MacroParser:
>
> - Fixes the above problem; now the execution of the "DEFINE" directives
> inside IFDEF/IFNDEF is correct.
Ok

> - Adds support for "local" definitions
Cool, thanks.

I've merged your fix with the current status of Camlp4MacroParser
which now handles signature items and a more general -D<uident>=<expr>
option.

I've applied it to the CVS (release310 branch) that is now available
from the public CVS repository.

Since some changes have been applied to camlp4 since OCaml-3.10+beta
release I suggest you to get the last version to do your porting work.

Best regard and many thanks,

-- 
Nicolas Pouillard


      reply	other threads:[~2007-04-17 19:28 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-16 20:20 3.10.0+beta - "Not_found" from a custom camlp4 executable (Delete.delete_rule_in_suffix) Aleksey Nogin
2007-04-16 20:26 ` [Caml-list] " Nicolas Pouillard
2007-04-16 22:49   ` 3.10.0+beta: pa_macro vs Camlp4MacroParser [Was: "Not_found" from a custom camlp4 executable] Aleksey Nogin
2007-04-17  1:12     ` 3.10.0+beta: Corrected and improved Camlp4MacroParser Aleksey Nogin
2007-04-17 19:28       ` Nicolas Pouillard [this message]

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=cd67f63a0704171228j15bbc76ie63db5c6c718c87c@mail.gmail.com \
    --to=nicolas.pouillard@gmail.com \
    --cc=caml-list@inria.fr \
    --cc=nogin@metaprl.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;
as well as URLs for NNTP newsgroup(s).