Re: 63-rename-var-vars.patch

[Tom Tromey]

>>>>> "Akim" == Akim Demaille <address@hidden> writes:

Akim> The aim of this patch and the next is to clarify the status of
Akim> variables: who owns them, and how their defined type (+=, := or
Akim> =) can change.

Why does it matter who owns them?

Akim> I fail to understand why the user is not allowed to += a
Akim> nonexistent variable.

Perhaps this was intended to catch user errors?

Actually in the automake I have we don't have this restriction.
We just have a check to make sure the user doesn't `+=' an
internally-defined variable.  This check was added because the
implementation didn't allow this to work.

We can and probably should change this.  It is a bad idea for a user
to do this, but ...

Akim>   * automake.in (%var_was_plus_eq): Rename as...
Akim>   (%var_type): this.
Akim>   (%def_type): Remove.
Akim>   (&variable_define): %var_type may now hold `', `+', or `:'.
Akim>   (%conditional, %am_vars, %content_lines): Rename as...
Akim>   (%var_value, %var_comment, %var_line): these.

Ok.

Tom