diff --git a/pixie.nimble b/pixie.nimble index ed00d2d..b96e0ab 100644 --- a/pixie.nimble +++ b/pixie.nimble @@ -1,4 +1,4 @@ -version = "0.0.16" +version = "0.0.17" author = "Andre von Houck and Ryan Oldenburg" description = "Full-featured 2d graphics library for Nim." license = "MIT" diff --git a/src/pixie/blends.nim b/src/pixie/blends.nim index 997f602..9c4e562 100644 --- a/src/pixie/blends.nim +++ b/src/pixie/blends.nim @@ -331,7 +331,7 @@ proc hardLight(backdrop, source: uint32): uint8 {.inline.} = else: screen(backdrop, 2 * source - 255) -proc blendNormal(backdrop, source: ColorRGBA): ColorRGBA = +proc blendNormal*(backdrop, source: ColorRGBA): ColorRGBA = result = source result = alphaFix(backdrop, source, result) diff --git a/src/pixie/fileformats/svg.nim b/src/pixie/fileformats/svg.nim index 14ac205..7ee438e 100644 --- a/src/pixie/fileformats/svg.nim +++ b/src/pixie/fileformats/svg.nim @@ -94,9 +94,7 @@ proc draw( return case node.tag: - of "title": - discard - of "desc": + of "title", "desc": discard of "g": @@ -110,12 +108,11 @@ proc draw( let d = node.attr("d") ctx = decodeCtx(ctxStack[^1], node) + path = parsePath(d) if ctx.fill != ColorRGBA(): - img.fillPath(d, ctx.fill, ctx.transform) + img.fillPath(path, ctx.fill, ctx.transform) if ctx.stroke != ColorRGBA() and ctx.strokeWidth > 0: - img.strokePath( - d, ctx.stroke, ctx.strokeWidth, ctx.transform - ) + img.strokePath(path, ctx.stroke, ctx.strokeWidth, ctx.transform) of "line": let @@ -125,7 +122,7 @@ proc draw( x2 = parseFloat(node.attr("x2")) y2 = parseFloat(node.attr("y2")) - let path = newPath() + var path: Path path.moveTo(x1, y1) path.lineTo(x2, y2) path.closePath() @@ -150,7 +147,7 @@ proc draw( if vecs.len == 0: failInvalid() - let path = newPath() + var path: Path path.moveTo(vecs[0]) for i in 1 ..< vecs.len: path.lineTo(vecs[i]) @@ -172,12 +169,8 @@ proc draw( width = parseFloat(node.attr("width")) height = parseFloat(node.attr("height")) - let path = newPath() - path.moveTo(x, y) - path.lineTo(x + width, y) - path.lineTo(x + width, y + height) - path.lineTo(x, y + height) - path.closePath() + var path: Path + path.rect(x, y, width, height) if ctx.fill != ColorRGBA(): img.fillPath(path, ctx.fill, ctx.transform) @@ -207,7 +200,7 @@ proc draw( magicX = (4.0 * (-1.0 + sqrt(2.0)) / 3) * rx magicY = (4.0 * (-1.0 + sqrt(2.0)) / 3) * ry - let path = newPath() + var path: Path path.moveTo(cx + rx, cy) path.bezierCurveTo(cx + rx, cy + magicY, cx + magicX, cy + ry, cx, cy + ry) path.bezierCurveTo(cx - magicX, cy + ry, cx - rx, cy + magicY, cx - rx, cy) @@ -215,13 +208,10 @@ proc draw( path.bezierCurveTo(cx + magicX, cy - ry, cx + rx, cy - magicY, cx + rx, cy) path.closePath() - let d = $path if ctx.fill != ColorRGBA(): - img.fillPath(d, ctx.fill, ctx.transform) + img.fillPath(path, ctx.fill, ctx.transform) if ctx.stroke != ColorRGBA() and ctx.strokeWidth > 0: - img.strokePath( - d, ctx.stroke, ctx.strokeWidth, ctx.transform - ) + img.strokePath(path, ctx.stroke, ctx.strokeWidth, ctx.transform) else: raise newException(PixieError, "Unsupported SVG tag: " & node.tag & ".") diff --git a/src/pixie/images.nim b/src/pixie/images.nim index d6523a9..d9f3b3d 100644 --- a/src/pixie/images.nim +++ b/src/pixie/images.nim @@ -88,7 +88,7 @@ proc fillUnsafe(data: var seq[ColorRGBA], rgba: ColorRGBA, start, len: int) = u32 = cast[uint32](rgba) u64 = cast[uint64]([u32, u32]) for j in countup(i, start + len - 2, 2): - cast[ptr uint64](image.data[j].addr)[] = u64 + cast[ptr uint64](data[j].addr)[] = u64 i += 2 # Fill whatever is left the slow way for j in i ..< start + len: @@ -172,11 +172,8 @@ proc magnifyBy2*(image: Image, scale2x: int): Image = proc magnifyBy2*(image: Image): Image = image.magnifyBy2(2) -when defined(release): - {.pop.} - proc toPremultipliedAlpha*(image: Image) = - ## Converts an image to premultiplied alpha from straight. + ## Converts an image to premultiplied alpha from straight alpha. var i: int when defined(amd64) and not defined(pixieNoSimd): # When supported, SIMD convert as much as possible @@ -190,12 +187,11 @@ proc toPremultipliedAlpha*(image: Image) = var color = mm_loadu_si128(image.data[j].addr) alpha = mm_and_si128(color, alphaMask) - alpha = mm_or_si128(alpha, mm_srli_epi32(alpha, 16)) - - var colorEven = mm_slli_epi16(color, 8) colorOdd = mm_and_si128(color, oddMask) + alpha = mm_or_si128(alpha, mm_srli_epi32(alpha, 16)) + colorEven = mm_mulhi_epu16(colorEven, alpha) colorOdd = mm_mulhi_epu16(colorOdd, alpha) @@ -228,6 +224,9 @@ proc toStraightAlpha*(image: Image) = c.g = ((c.g.uint32 * multiplier) div 255).uint8 c.b = ((c.b.uint32 * multiplier) div 255).uint8 +when defined(release): + {.pop.} + proc draw*(a, b: Image, mat: Mat3, blendMode = bmNormal) proc draw*(a, b: Image, pos = vec2(0, 0), blendMode = bmNormal) {.inline.} diff --git a/src/pixie/paths.nim b/src/pixie/paths.nim index 34829b7..f1ccc3d 100644 --- a/src/pixie/paths.nim +++ b/src/pixie/paths.nim @@ -16,12 +16,9 @@ type kind*: PathCommandKind numbers*: seq[float32] - Path* = ref object - at*: Vec2 + Path* = object commands*: seq[PathCommand] - -proc newPath*(): Path = - result = Path() + start, at: Vec2 # Maintained by moveTo, lineTo, etc. Used by arcTo. proc parameterCount(kind: PathCommandKind): int = case kind: @@ -35,7 +32,6 @@ proc parameterCount(kind: PathCommandKind): int = proc parsePath*(path: string): Path = ## Converts a SVG style path into seq of commands. - result = newPath() if path.len == 0: return @@ -188,7 +184,7 @@ proc `$`*(path: Path): string = if i != path.commands.len - 1 or j != command.numbers.len - 1: result.add " " -proc transform*(path: Path, mat: Mat3) = +proc transform*(path: var Path, mat: Mat3) = for command in path.commands.mitems: case command.kind: of Close: @@ -242,6 +238,130 @@ proc transform*(path: Path, mat: Mat3) = command.numbers[5] = to.x command.numbers[6] = to.y +proc addPath*(path: var Path, other: Path) = + ## Adds a path to the current path. + path.commands.add(other.commands) + +proc closePath*(path: var Path) = + path.commands.add(PathCommand(kind: Close)) + path.at = path.start + +proc moveTo*(path: var Path, x, y: float32) = + path.commands.add(PathCommand(kind: Move, numbers: @[x, y])) + path.start = vec2(x, y) + path.at = path.start + +proc moveTo*(path: var Path, v: Vec2) {.inline.} = + path.moveTo(v.x, v.y) + +proc lineTo*(path: var Path, x, y: float32) = + path.commands.add(PathCommand(kind: Line, numbers: @[x, y])) + path.at = vec2(x, y) + +proc lineTo*(path: var Path, v: Vec2) {.inline.} = + path.lineTo(v.x, v.y) + +proc bezierCurveTo*(path: var Path, x1, y1, x2, y2, x3, y3: float32) = + ## Adds a cubic Bézier curve to the path. This requires three points. + ## The first two points are control points and the third is the end point. + ## The starting point is the last point in the current path, which can be + ## changed using moveTo() before creating the curve. + path.commands.add(PathCommand( + kind: Cubic, + numbers: @[x1, y1, x2, y2, x3, y3] + )) + path.at = vec2(x3, y3) + +proc bezierCurveTo*(path: var Path, ctrl1, ctrl2, to: Vec2) {.inline.} = + path.bezierCurveTo(ctrl1.x, ctrl1.y, ctrl2.x, ctrl2.y, to.x, to.y) + +proc quadraticCurveTo*(path: var Path, x1, y1, x2, y2: float32) = + ## Adds a quadratic Bézier curve to the path. This requires 2 points. + ## The first point is the control point and the second is the end point. + ## The starting point is the last point in the current path, which can be + ## changed using moveTo() before creating the curve. + path.commands.add(PathCommand( + kind: Quad, + numbers: @[x1, y1, x2, y2] + )) + path.at = vec2(x2, y2) + +proc quadraticCurveTo*(path: var Path, ctrl, to: Vec2) {.inline.} = + path.quadraticCurveTo(ctrl.x, ctrl.y, to.x, to.y) + +proc arcTo*(path: var Path, ctrl1, ctrl2: Vec2, radius: float32) {.inline.} = + ## Adds a circular arc to the current sub-path, using the given control + ## points and radius. + + const epsilon = 1e-6.float32 + + var radius = radius + if radius < 0: + radius = -radius + + if path.commands.len == 0: + path.moveTo(ctrl1) + + let + a = path.at - ctrl1 + b = ctrl2 - ctrl1 + + if a.lengthSq() < epsilon: + # If the control point is coincident with at, do nothing + discard + elif abs(a.y * b.x - a.x * b.y) < epsilon or radius == 0: + # If ctrl1, a and b are colinear or coincident or radius is zero + path.lineTo(ctrl1) + else: + let + c = ctrl2 - path.at + als = a.lengthSq() + bls = b.lengthSq() + cls = c.lengthSq() + al = a.length() + bl = b.length() + l = radius * tan((PI - arccos((als + bls - cls) / 2 * al * bl)) / 2) + ta = l / al + tb = l / bl + + if abs(ta - 1) > epsilon: + # If the start tangent is not coincident with path.at + path.lineTo(ctrl1 + a * ta) + + let to = ctrl1 + b * tb + path.commands.add(PathCommand( + kind: Arc, + numbers: @[ + radius, + radius, + 0, + 0, + if a.y * c.x > a.x * c.y: 1 else: 0, + to.x, + to.y + ] + )) + path.at = to + +proc arcTo*(path: var Path, x1, y1, x2, y2, radius: float32) = + path.arcTo(vec2(x1, y1), vec2(x2, y2), radius) + +proc rect*(path: var Path, x, y, w, h: float32) = + path.moveTo(x, y) + path.lineTo(x + w, y) + path.lineTo(x + w, y + h) + path.lineTo(x, y + h) + path.closePath() + +proc polygon*(path: var Path, x, y, size: float32, sides: int) = + ## Draws a n sided regular polygon at (x, y) with size. + path.moveTo(x + size * cos(0.0), y + size * sin(0.0)) + for side in 0 .. sides: + path.lineTo( + x + size * cos(side.float32 * 2.0 * PI / sides.float32), + y + size * sin(side.float32 * 2.0 * PI / sides.float32) + ) + proc commandsToPolygons*(commands: seq[PathCommand]): seq[seq[Vec2]] = ## Converts SVG-like commands to simpler polygon @@ -664,7 +784,6 @@ proc fillPolygons*( polys: seq[seq[Vec2]], color: ColorRGBA, windingRule: WindingRule, - blendMode: BlendMode = bmNormal, quality = 4 ) = var bounds = newSeq[Rect](polys.len) @@ -672,7 +791,6 @@ proc fillPolygons*( bounds[i] = computeBounds(poly) const ep = 0.0001 * PI - let mixer = blendMode.mixer() proc scanLineHits( polys: seq[seq[Vec2]], @@ -755,7 +873,7 @@ proc fillPolygons*( var colorWithAlpha = color colorWithAlpha.a = uint8(a * 255.0) let rgba = image.getRgbaUnsafe(x, y) - image.setRgbaUnsafe(x, y, mixer(rgba, colorWithAlpha)) + image.setRgbaUnsafe(x, y, blendNormal(rgba, colorWithAlpha)) {.pop.} @@ -865,137 +983,3 @@ proc strokePath*( for i, p in poly.mpairs: poly[i] = mat * p image.fillPolygons(image.wh, polys2, color, windingRule) - -proc addPath*(path: Path, other: Path) = - ## Adds a path to the current path. - path.commands &= other.commands - -proc closePath*(path: Path) = - ## Causes the point of the pen to move back to the start of the current sub-path. It tries to draw a straight line from the current point to the start. If the shape has already been closed or has only one point, this function does nothing. - path.commands.add PathCommand(kind: Close) - -proc moveTo*(path: Path, x, y: float32) = - ## Moves the starting point of a new sub-path to the (x, y) coordinates. - path.commands.add PathCommand(kind: Move, numbers: @[x, y]) - path.at = vec2(x, y) - -proc moveTo*(path: Path, pos: Vec2) = - path.moveTo(pos.x, pos.y) - -proc lineTo*(path: Path, x, y: float32) = - ## Connects the last point in the subpath to the (x, y) coordinates with a straight line. - path.commands.add PathCommand(kind: Line, numbers: @[x, y]) - path.at = vec2(x, y) - -proc lineTo*(path: Path, pos: Vec2) = - path.lineTo(pos.x, pos.y) - -proc bezierCurveTo*(path: Path, x1, y1, x2, y2, x3, y3: float32) = - ## Adds a cubic Bézier curve to the path. It requires three points. The first two points are control points and the third one is the end point. The starting point is the last point in the current path, which can be changed using moveTo() before creating the Bézier curve. - path.commands.add(PathCommand(kind: Cubic, numbers: @[ - x1, y1, x2, y2, x3, y3 - ])) - -proc quadraticCurveTo*(path: var Path, x1, y1, x2, y2: float32) = - ## Adds a quadratic Bézier curve to the path. This requires 2 points. - ## The first point is the control point and the second is the end point. - ## The starting point is the last point in the current path, which can be - ## changed using moveTo() before creating the curve. - path.commands.add(PathCommand( - kind: Quad, - numbers: @[x1, y1, x2, y2] - )) - path.at = vec2(x2, y2) - -proc quadraticCurveTo*(path: var Path, ctrl, to: Vec2) {.inline.} = - path.quadraticCurveTo(ctrl.x, ctrl.y, to.x, to.y) - -proc arc*(path: Path) = - ## Adds an arc to the path which is centered at (x, y) position with radius r starting at startAngle and ending at endAngle going in the given direction by anticlockwise (defaulting to clockwise). - raise newException(ValueError, "not implemented") - -proc arcTo*(path: Path, x1, y1, x2, y2, r: float32) = - ## Adds a circular arc to the path with the given control points and radius, connected to the previous point by a straight line. - - const epsilon = 1e-6.float32 - - if path.commands.len == 0: - # Is this path empty? Move to (x1,y1). - path.moveTo(x1, y1) - - var r = r - if r < 0: - # Is the radius negative? Flip it. - r = -r - - var - x21 = x2 - x1 - y21 = y2 - y1 - x01 = path.at.x - x1 - y01 = path.at.y - y1 - l01_2 = x01 * x01 + y01 * y01 - - if not (l01_2 > epsilon): - # Is (x1,y1) coincident with (x0,y0)? Do nothing. - discard - - elif not (abs(y01 * x21 - y21 * x01) > epsilon) or r == 0: - # // Or, are (x0,y0), (x1,y1) and (x2,y2) colinear? - # // Equivalently, is (x1,y1) coincident with (x2,y2)? - # // Or, is the radius zero? Line to (x1,y1). - - path.lineTo(x1, y1) - - else: - # Draw an arc - var - x20 = x2 - path.at.x - y20 = y2 - path.at.y - l21_2 = x21 * x21 + y21 * y21 - l20_2 = x20 * x20 + y20 * y20 - l21 = sqrt(l21_2) - l01 = sqrt(l01_2) - l = r * tan((PI - arccos((l21_2 + l01_2 - l20_2) / (2 * l21 * l01))) / 2).float32 - t01 = l / l01 - t21 = l / l21 - - # If the start tangent is not coincident with path.at, line to. - if abs(t01 - 1) > epsilon: - path.lineTo(x1 + t01 * x01, y1 + t01 * y01) - - path.at.x = x1 + t21 * x21 - path.at.y = y1 + t21 * y21 - path.commands.add(PathCommand( - kind: Arc, - numbers: @[ - r, - r, - 0, - 0, - if y01 * x20 > x01 * y20: 1 else: 0, - path.at.x, - path.at.y - ] - )) - -proc ellipse*(path: Path) = - ## Adds an elliptical arc to the path which is centered at (x, y) position with the radii radiusX and radiusY starting at startAngle and ending at endAngle going in the given direction by anticlockwise (defaulting to clockwise). - raise newException(ValueError, "not implemented") - -proc rect*(path: Path, x, y, w, h: float32) = - ## Creates a path for a rectangle at position (x, y) with a size that is determined by width and height. - path.moveTo(x, y) - path.lineTo(x+w, y) - path.lineTo(x+w, y+h) - path.lineTo(x, y+h) - path.lineTo(x, y) - path.closePath() - -proc polygon*(path: Path, x, y, size: float32, sides: int) = - ## Draws a n sided regular polygon at (x, y) with size. - path.moveTo(x + size * cos(0.0), y + size * sin(0.0)) - for side in 0 .. sides: - path.lineTo( - x + size * cos(side.float32 * 2.0 * PI / sides.float32), - y + size * sin(side.float32 * 2.0 * PI / sides.float32) - ) diff --git a/tests/test_paths.nim b/tests/test_paths.nim index 1cb9e78..ad22f1e 100644 --- a/tests/test_paths.nim +++ b/tests/test_paths.nim @@ -90,7 +90,7 @@ block: block: let image = newImage(100, 100) - var path = newPath() + var path: Path path.moveTo(10, 10) path.lineTo(10, 90) path.lineTo(90, 90) @@ -149,14 +149,14 @@ block: image.writeFile("tests/images/paths/pathCornerArc.png") block: - let image = newImage(100, 100) - var path = newPath() let + image = newImage(100, 100) r = 10.0 x = 10.0 y = 10.0 h = 80.0 w = 80.0 + var path: Path path.moveTo(x+r, y) path.arcTo(x+w, y, x+w, y+h, r) path.arcTo(x+w, y+h, x, y+h, r)