highestVisibleXIndex is smaller than lowestVisibleXIndex

[lionhylra]

I just downloaded the demo app, adding one line _chartView.autoScaleMinMaxEnabled = YES; to the BarChartViewController.m at line:56.

And I run the demo app, and open the bar chart, zoom in. It crashes.
The screenshot:
screenshot
Back trace:
back trace

[lionhylra]

I’ve added back trace.

On 12 Apr 2016, at 5:59 PM, Xuan notifications@github.com wrote:

send out the back trace, so we know where it crashes.

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub #940 (comment)

[liuxuan30]

you can simply post the text rather than image.
It crashes at BarChartDataSet.calcMinMax(), not highestVisibleXIndex is smaller than lowestVisibleXIndex
I'm adding a fix soon. It's simple, just use stride

[lionhylra]

Yes, exactly. And in my own project, when I apply 'autoScaleMinMaxEnabled = true' to my bar chart, it crashes directly for the same reason.

[lionhylra]

In the screenshot above, the end value is smaller than start value in the for loop.

[lionhylra]

I don't think changing 'for in range' to 'stride' will fix

[liuxuan30]

why not? I did a quick test, it won't crash then.

[liuxuan30]

        _lastStart = start
        _lastEnd = end

        _yMin = yMin(entries[start])
        _yMax = yMax(entries[start])

        for i in start.stride(through: endValue, by: 1)
        {

if endValue < start, we won't step in the loop, and yMin, yMax is from entries[start]. So I think it is harmless.

[lionhylra]

ok, I will try it later. Thanks