build with -Werror (5ba225a9) · Commits · github_fork / Clip

CMakeLists.txt

+1 −0

Original line number	Diff line number	Diff line
		@@ -4,6 +4,7 @@ enable_testing()

		set(CMAKE_MODULE_PATH ${CMAKE_MODULE_PATH} "${CMAKE_CURRENT_SOURCE_DIR}/extra/cmake")
		set(CMAKE_CXX_STANDARD 17)
		set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Werror")

		include_directories(${CMAKE_CURRENT_SOURCE_DIR}/common)
		include_directories(${CMAKE_CURRENT_SOURCE_DIR}/common/utils)

+2 −0

Original line number	Diff line number	Diff line
		@@ -128,6 +128,8 @@ std::string svg_path_data(const Path& path) {
		path_data << StringUtil::format("a$0 $0 0 1 0 $1 0 ", cmd[2], cmd[2] * 2);
		path_data << StringUtil::format("a$0 $0 0 1 0 $1 0 ", cmd[2], -cmd[2] * 2);
		break;
		default:
		break; // not yet implemented
		}
		}

+4 −0

Original line number	Diff line number	Diff line
		@@ -93,6 +93,8 @@ Status Rasterizer::fillPath(const layer_ops::BrushFillOp& op) {
		case PathCommand::ARC_TO:
		cairo_arc(cr_ctx, cmd[0], cmd[1], cmd[2], cmd[3], cmd[4]);
		break;
		default:
		break; // not yet implemented
		}
		}

		@@ -134,6 +136,8 @@ Status Rasterizer::strokePath(const layer_ops::BrushStrokeOp& op) {
		case PathCommand::ARC_TO:
		cairo_arc(cr_ctx, cmd[0], cmd[1], cmd[2], cmd[3], cmd[4]);
		break;
		default:
		break; // not yet implemented
		}
		}

+19 −0

Original line number	Diff line number	Diff line
		@@ -257,6 +257,9 @@ Status renderAxis(
		clip.x + clip.w,
		frame);
		break;
		case AxisPosition::CENTER_HORIZ:
		case AxisPosition::CENTER_VERT:
		return ERROR_NOT_IMPLEMENTED;
		}

		return rc;
		@@ -284,6 +287,9 @@ ReturnCode axis_expand_auto(
		case AxisPosition::LEFT:
		out->tick_position = AxisLabelPosition::LEFT;
		break;
		case AxisPosition::CENTER_HORIZ:
		case AxisPosition::CENTER_VERT:
		return ERROR_NOT_IMPLEMENTED;
		}
		break;
		case AxisLabelPosition::INSIDE:
		@@ -300,8 +306,13 @@ ReturnCode axis_expand_auto(
		case AxisPosition::LEFT:
		out->tick_position = AxisLabelPosition::RIGHT;
		break;
		case AxisPosition::CENTER_HORIZ:
		case AxisPosition::CENTER_VERT:
		return ERROR_NOT_IMPLEMENTED;
		}
		break;
		default:
		break;
		};

		switch (out->label_position) {
		@@ -319,6 +330,9 @@ ReturnCode axis_expand_auto(
		case AxisPosition::LEFT:
		out->label_position = AxisLabelPosition::LEFT;
		break;
		case AxisPosition::CENTER_HORIZ:
		case AxisPosition::CENTER_VERT:
		return ERROR_NOT_IMPLEMENTED;
		}
		break;
		case AxisLabelPosition::INSIDE:
		@@ -335,8 +349,13 @@ ReturnCode axis_expand_auto(
		case AxisPosition::LEFT:
		out->label_position = AxisLabelPosition::RIGHT;
		break;
		case AxisPosition::CENTER_HORIZ:
		case AxisPosition::CENTER_VERT:
		return ERROR_NOT_IMPLEMENTED;
		}
		break;
		default:
		break;
		};

		if (in.label_placement) {