-
-
Notifications
You must be signed in to change notification settings - Fork 6k
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
Fix Type 'ChartDataSet' does not conform to protocol 'RangeReplaceabl… #4881
Conversation
Can somebody merge this PR? iOS 16 is near and this error breaks my app. Thanks! |
@danielgindi I think this one is top priority if you don't mind |
This issue is also happening on 3.6.0 with Xcode 14 beta 6. Will merging this fix also fix it for people not on version 4.x.x? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The change works but I'd rather not change the Range
class's name.
@@ -571,4 +571,12 @@ extension ChartDataSet: RangeReplaceableCollection { | |||
entries.removeAll(keepingCapacity: keepCapacity) | |||
notifyDataSetChanged() | |||
} | |||
|
|||
public func replaceSubrange<C>( | |||
_ subrange: Range<Index>, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think if we just use Swift.Range<Index>
here we don't have to rename the other Range class. I think it would be a better fix for the problem.
Close in favour of #486. Thanks for the contribution. |
#486 is seven years old and unrelated. I presume that superseding issue number is a typo? |
Ahhhh sorry about that it was #4861 |
For some reason I got this error on xCode - beta.
Fix implemented by: