New crater reports: 1.1 stable vs beta-2015-07-10 and nightly-2015-07-10

[brson]

Here’s stable vs. beta: https://gist.github.com/brson/e4d8184be1d9af0f901a

And here’s stable vs. nightly: https://gist.github.com/brson/9362098f0ca2ef38d1c2

Beta has 4 root regressions vs. stable and nightly has 24 root regressions vs. stable.

Here are the beta regressions:

• libnetfilter_queue-0.0.3 (before) (after)
• netfilter_queue-0.2.1 (before) (after)
• rust_events-0.8.1 (before) (after)
• simple_stats-0.0.4 (before) (after)

And nightly:

• protobuf-1.0.1 (before) (after)
• ncollide_queries-0.2.2 (before) (after)
• nickel-0.5.0 (before) (after)
• startuppong-0.2.0 (before) (after)
• timely-0.0.6 (before) (after)
• hueclient-0.3.0 (before) (after)
• coinbaser-0.1.0 (before) (after)
• libnetfilter_queue-0.0.3 (before) (after)
• netfilter_queue-0.2.1 (before) (after)
• dcpu16-gui-0.0.2 (before) (after)
• piston3d-gfx_voxel-0.1.0 (before) (after)
• piston_window_game-0.1.0 (before) (after)
• dev_menu-0.1.0 (before) (after)
• poglgame-0.4.0 (before) (after)
• recaptcha-0.1.0 (before) (after)
• rust-cleverbot-0.1.1 (before) (after)
• rust_events-0.8.1 (before) (after)
• rustfm-0.0.3 (before) (after)
• rustful-0.2.2 (before) (after)
• shiplift-0.1.2 (before) (after)
• simple_stats-0.0.4 (before) (after)
• strava-0.1.1 (before) (after)
• tba-0.0.0 (before) (after)
• webdriver-0.2.0 (before) (after)

Analysis - stable vs. beta

3 of the beta regressions are because #[packed] is no longer accepted. Looks like it was probably cause by this PR. It needs to be either fixed or mentioned in the relnotes as a breaking change.

The 4th beta regression though is a type inference failure. Somebody please investigate.

Analysis stable vs. nightly

• ‘missing lifetime specifier’ - protobuf
• ‘type annotations required’ - ncollide_queries
• ‘missing documentation’ - nickel, startuppong, hueclient, coinbaser, dcpu16-gui, piston32-gfx_voxel, piston_window_game, dev_menu, poglgame, recaptcha, rust-cleverbot, rustfmt, rustful, shiplift, strava, tba, webdriver
• ‘type of value must be known’ - timely, simple_stats
• #[packed] - libnetfilter_queue, netfilter_queue
• ‘illegal cast: cast through raw pointer first’ - rust_events

I don’t know what caused any of these.

[brson]

The immediate question we need to answer is whether to do anything about the #[packed] regression on beta (this PR). We might restore the previous behavior of #[packed] and leave it deprecated.

It would also be delightful if somebody could figure out the other beta regression and start figuring out the nightly regressions.

[nikomatsakis]

missing documentation

This at least is the improvements to the missing doc lint, and seems to suggest the importance of:

https://github.com/rust-lang/rfcs/pull/1193

[bluss]

‘type of value must be known’ - timely, simple_stats

[ Since timely does not break in beta, it can’t be just Extend breaking it – timely’s issue is #26952 ]

These both look like they come from the addition of impl Extend<&T> for Vec<T> (and other collections. With both Extend<T> and Extend<&T> now, type inference is now sometimes ambiguous. Issue #26279

[bluss]

‘type annotations required’ - ncollide_queries

This crate is already updated and I assume it is fixed. The author says “Type inference regression?”

Edit: Maybe that projections can’t drive inference anymore, could it be that? Though the time is wrong, that landed in nightly weeks ago.

[nikomatsakis]

This crate is already updated and I assume it is fixed. The author says “Type inference regression?”

I found the cause of the problem. I’ll be opening a PR soon.

[system]

This topic was automatically closed 90 days after the last reply. New replies are no longer allowed.