Code review comment for lp:~maddevelopers/mg5amcnlo/1.5.8

Revision history for this message
Olivier Mattelaer (olivier-mattelaer) wrote :

I pushed it right after sending the message (This is not efficient so why keep those ;-) )

Cheers,

Olivier
On Mar 4, 2013, at 8:10 AM, Johan Alwall <email address hidden> wrote:

> Hi Olivier,
>
>> An Empty line should ALWAYS be allowed!
>> (even if this might not be optimal for the size of the file)
>
> Hey, don't shoot the pianist! Remember that we have dozens of lhe file
> readers in as many programs, so you can't expect all of them to work on
> *anything* outside of the standard. I'm pretty sure an empty line is not
> among the things described in the standard to be allowed. So it needs to go.
>
> Cheers,
> Johan
>
> --
> https://code.launchpad.net/~maddevelopers/madgraph5/1.5.8/+merge/150437
> You proposed lp:~maddevelopers/madgraph5/1.5.8 for merging.

« Back to merge proposal