enable interpreter for darwin

Tom Tromey tromey@redhat.com
Fri Mar 8 08:37:00 GMT 2002


>>>>> "Andreas" == Andreas Tobler <toa@pop.agri.ch> writes:

> 2002-03-06		Andreas Tobler <a.tobler@schweiz.ch>
>     * configure.host: Enable the interpreter for Darwin PPC by default.
>                       This is now possible since libffi offers the neccessary 
>                       functionality.
        
I would probably write it as:

2002-03-06  Andreas Tobler <a.tobler@schweiz.ch>

	* configure.host: Enable the interpreter for Darwin PPC by default.

You don't need the explanation.  Or, if you do need it, it should go
in the source code and not the ChangeLog.  In this case I think things
are fine without it.  Anybody seeing that the interpreter is enabled
will know that libffi works on this platform.


Or maybe even:

	* configure.host (powerpc-*-darwin*): Enable interpreter.

The idea here is that the case statement is the body of this file, and
so each branch is sort of a function-like thing.  So you could use the
function-style syntax.


Other nits:
Use a tab before the `*'.
If you have lines after the first line, all the lines should line up
(don't indent subsequent lines more).

HTH,
Tom



More information about the Java-patches mailing list