Skip to content

[SR-14761] Improve diagnostic for implicitly overridden init() #38365

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

Merged

Conversation

HannaYakusevych
Copy link
Contributor

@HannaYakusevych HannaYakusevych commented Jul 13, 2021

Provide more detailed diagnostic message when implicitly synthesized @objc initializer in subclass was overridden inside the subclass' extension

Resolves SR-14761

@xedin xedin requested a review from CodaFi July 13, 2021 16:51
Copy link
Contributor

@xedin xedin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense to me. @CodaFi Could you please take a look?

Provide more detailed diagnostic message when implicitly synthesized @objc initializer in subclass was overridden inside the subclass' extension

Resolves: rdar://problem/79179597
@HannaYakusevych HannaYakusevych force-pushed the fix-diagnostic-implicitly-overriden-init branch from d23bb3e to c0ea9b2 Compare July 13, 2021 23:10
@CodaFi
Copy link
Contributor

CodaFi commented Jul 14, 2021

Looks good

@swift-ci smoke test

@xedin
Copy link
Contributor

xedin commented Jul 14, 2021

@swift-ci please smoke test Linux platform

@xedin
Copy link
Contributor

xedin commented Jul 14, 2021

@swift-ci please clean smoke test Linux platform

1 similar comment
@xedin
Copy link
Contributor

xedin commented Jul 14, 2021

@swift-ci please clean smoke test Linux platform

@xedin xedin changed the title [SE-14761] Changed diagnostic for implicitly overridden init() [SR-14761] Improve diagnostic for implicitly overridden init() Jul 14, 2021
@xedin
Copy link
Contributor

xedin commented Jul 15, 2021

@swift-ci please clean smoke test Linux platform

@xedin
Copy link
Contributor

xedin commented Jul 15, 2021

@swift-ci please smoke test Linux platform

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.

3 participants