fix for #8088

[minhnhdo]

fix for #8088, along with a test.

[Aatch]

I don't think this is the correct fix. That syntax is very old and there is newer, obsolete, syntax that doesn't have any obsolete detection. Just deleting the obsolete syntax detection for it is fine.

[kemurphy]

== @Aatch

[minhnhdo]

@kemurphy there exists a compile-fail test for this already. It's in src/test/compile-fail/obsolete.rs.

// ...
struct s {
    let foo: (),
    //~^ ERROR obsolete syntax: `let` in field declaration
    bar: ();
    //~^ ERROR obsolete syntax: field declaration terminated with semicolon
    new() { }
    //~^ ERROR obsolete syntax: struct constructor
}
// ...

[minhnhdo]

@Aatch so you also want to just delete the pieces of code that deals with this?

[kemurphy]

Yes, just cut it away entirely.

[minhnhdo]

@kemurphy and the test also?

[alexcrichton]

r+, man that is some seriously ancient syntax. Thanks!

[kemurphy]

Looks good to me.