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

Jump to bottom

[SR-4518] Remote mirrors needs to support multi-payload enums #47095

Closed
swift-ci opened this issue Apr 6, 2017 · 4 comments
Closed

[SR-4518] Remote mirrors needs to support multi-payload enums #47095

swift-ci opened this issue Apr 6, 2017 · 4 comments
Assignees
@mikeash
Labels
bug A deviation from expected or documented behavior. Also: expected but undesirable behavior. compiler The Swift compiler in itself memory leak bug: Memory leak

Comments

@swift-ci
Copy link
Collaborator

swift-ci commented Apr 6, 2017

Previous ID SR-4518
Radar rdar://31666592
Original Reporter Jay18001 (JIRA User)
Type Bug

Attachment: Download

Environment

I've only tested on iOS 10.3 with Xcode 8.3 on 10.12.3 (16D32)

Additional Detail from JIRA
Votes 0
Component/s Compiler
Labels Bug, Leak
Assignee @mikeash
Priority Medium

md5: d6c09cb4c02ae76ce77a67ef1ead510e

Issue Description:

There is a swift_slowAlloc leak when using an enum like the following. The leak only occurs when the second case is used in the third like auto(estimate: .fixed(size: 10)).

public indirect enum Enum {
case ratio(ratio: CGFloat)
 case fixed(size: CGFloat)
 case auto(estimate: Enum)
}

However it does not accure when the first case is used with the third like auto(estimate: .ratio(ratio: 10))
@belkadan
Copy link
Contributor

belkadan commented Apr 7, 2017

I think this is a (known) bug in our leaks tool rather than an actual leak; looping the offending line of code forever did not seem to increase memory usage in a quick test. @jckarter?

@jckarter
Copy link
Member

jckarter commented Apr 7, 2017

@slavapestov would be better able to comment on that. IIRC some of those issues got fixed in iOS 10.3, but according to this bug this one is still happening in 10.3.

@slavapestov
Copy link
Member

The remote mirrors library does not support multi-payload enums yet unfortunately.

@swift-ci swift-ci transferred this issue from apple/swift-issues Apr 25, 2022
@slavapestov
Copy link
Member

Looks like this was fixed in Swift 5.7.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mikeash mikeash

Labels
bug A deviation from expected or documented behavior. Also: expected but undesirable behavior. compiler The Swift compiler in itself memory leak bug: Memory leak
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@slavapestov @jckarter @belkadan @swift-ci