Repository: danielgindi/Charts
File: Source/Charts/Data/Implementations/Standard/PieChartDataSet.swift (Swift)

Code before:
            if space > 20.0
            {
                space = 20.0
            }
            if space < 0.0
            {
                space = 0.0
            }
            _sliceSpace = space
        }
    }

    /// When enabled, slice spacing will be 0.0 when the smallest value is going to be smaller than the slice spacing itself.
    open var automaticallyDisableSliceSpacing: Bool = false

    /// indicates the selection distance of a pie slice
    open var selectionShift = CGFloat(18.0)

    open var xValuePosition: ValuePosition = .insideSlice
    open var yValuePosition: ValuePosition = .insideSlice

    /// When valuePosition is OutsideSlice, indicates line color
    open var valueLineColor: NSUIColor? = NSUIColor.black

    /// When valuePosition is OutsideSlice and enabled, line will have the same color as the slice
    open var valueLineAutoColor: Bool = false

    /// When valuePosition is OutsideSlice, indicates line width
    open var valueLineWidth: CGFloat = 1.0

    /// When valuePosition is OutsideSlice, indicates offset as percentage out of the slice size
    open var valueLinePart1OffsetPercentage: CGFloat = 0.75

    /// When valuePosition is OutsideSlice, indicates length of first half of the line
    open var valueLinePart1Length: CGFloat = 0.3

    /// When valuePosition is OutsideSlice, indicates length of second half of the line
    open var valueLinePart2Length: CGFloat = 0.4

    /// When valuePosition is OutsideSlice, this allows variable line length
    open var valueLineVariableLength: Bool = true

    /// the font for the slice-text labels
    open var entryLabelFont: NSUIFont? = nil

    /// the color for the slice-text labels
    open var entryLabelColor: NSUIColor? = nil

    /// the color for the highlighted sector
    open var highlightColor: NSUIColor? = nil


Review comment (suggestion):
Hi there, can you change it to `useValueColorForLine`? the name does not properly imply it's bool. If you can change it, it will be easier for me to merge this week. Otherwise I have to create another commit after yours.

Code after:
            if space > 20.0
            {
                space = 20.0
            }
            if space < 0.0
            {
                space = 0.0
            }
            _sliceSpace = space
        }
    }

    /// When enabled, slice spacing will be 0.0 when the smallest value is going to be smaller than the slice spacing itself.
    open var automaticallyDisableSliceSpacing: Bool = false

    /// indicates the selection distance of a pie slice
    open var selectionShift = CGFloat(18.0)

    open var xValuePosition: ValuePosition = .insideSlice
    open var yValuePosition: ValuePosition = .insideSlice

    /// When valuePosition is OutsideSlice, indicates line color
    open var valueLineColor: NSUIColor? = NSUIColor.black

    /// When valuePosition is OutsideSlice and enabled, line will have the same color as the slice
    open var useValueColorForLine: Bool = false

    /// When valuePosition is OutsideSlice, indicates line width
    open var valueLineWidth: CGFloat = 1.0

    /// When valuePosition is OutsideSlice, indicates offset as percentage out of the slice size
    open var valueLinePart1OffsetPercentage: CGFloat = 0.75

    /// When valuePosition is OutsideSlice, indicates length of first half of the line
    open var valueLinePart1Length: CGFloat = 0.3

    /// When valuePosition is OutsideSlice, indicates length of second half of the line
    open var valueLinePart2Length: CGFloat = 0.4

    /// When valuePosition is OutsideSlice, this allows variable line length
    open var valueLineVariableLength: Bool = true

    /// the font for the slice-text labels
    open var entryLabelFont: NSUIFont? = nil

    /// the color for the slice-text labels
    open var entryLabelColor: NSUIColor? = nil

    /// the color for the highlighted sector
    open var highlightColor: NSUIColor? = nil