Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change assert into if statement #63817

Closed
wants to merge 1 commit into from

Conversation

Nashenas88
Copy link
Contributor

Fixes #63765

I'm not sure if this is actually what should be done. It's not clear to me why the output of the Paren expression would be modified and why that would make sense.

If this does make sense, I'd like to add comments explaining what's actually occurring here so it's clear for the next person who has to update this area.

@Nashenas88
Copy link
Contributor Author

This should also have a test associated with it. Not sure on the exact code in polonius-engine that causes this case to come up. I was only able to narrow it down to the Func generic param in:

    fn filter_with<'leap, Tuple: Ord, Func: Fn(&Tuple) -> (Key, Val)>(
        &'leap self,
        key_func: Func,
    ) -> filter_with::FilterWith<'leap, Key, Val, Tuple, Func>
    where
        Key: 'leap,
        Val: 'leap;

(from datafrog)

@Nashenas88
Copy link
Contributor Author

This now works in the nightly compiler without my change. Going to close this PR.

@Nashenas88 Nashenas88 closed this Aug 22, 2019
@Nashenas88 Nashenas88 deleted the rustdoc_crash_63765 branch August 24, 2019 18:32
@Nashenas88 Nashenas88 restored the rustdoc_crash_63765 branch August 24, 2019 19:19
@Nashenas88
Copy link
Contributor Author

@GuillaumeGomez @Mark-Simulacrum can this be reopened? I shouldn't have closed this.

@Mark-Simulacrum
Copy link
Member

You'll need to open a new PR I believe - GitHub doesn't allow reopening pull requests where the underlying branch changed in-between.

@GuillaumeGomez
Copy link
Member

@Mark-Simulacrum TIL haha

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Panic while running rustdoc on polonius-engine
3 participants