diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-07-16 10:20:06 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-07-16 13:35:30 +0200 |
commit | 72fecbbc81bf375e2a5617db5c6e88a8a3bacb26 (patch) | |
tree | a875d573a8e5fb0a33ac1ed8a1a540e32033485e /include/basegfx | |
parent | 3da20d39e2cbda4d4c01ad206668924ab8c3f2b6 (diff) |
loplugin:mergeclasses merge BPixelRaster into BZPixelRaster
Change-Id: Ib6f7bfd2392daa5512d5fa68b69428ae9c8876f9
Reviewed-on: https://gerrit.libreoffice.org/57488
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'include/basegfx')
-rw-r--r-- | include/basegfx/raster/bpixelraster.hxx | 78 | ||||
-rw-r--r-- | include/basegfx/raster/bzpixelraster.hxx | 60 |
2 files changed, 41 insertions, 97 deletions
diff --git a/include/basegfx/raster/bpixelraster.hxx b/include/basegfx/raster/bpixelraster.hxx deleted file mode 100644 index b3f168754a8f..000000000000 --- a/include/basegfx/raster/bpixelraster.hxx +++ /dev/null @@ -1,78 +0,0 @@ -/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */ -/* - * This file is part of the LibreOffice project. - * - * This Source Code Form is subject to the terms of the Mozilla Public - * License, v. 2.0. If a copy of the MPL was not distributed with this - * file, You can obtain one at http://mozilla.org/MPL/2.0/. - * - * This file incorporates work covered by the following license notice: - * - * Licensed to the Apache Software Foundation (ASF) under one or more - * contributor license agreements. See the NOTICE file distributed - * with this work for additional information regarding copyright - * ownership. The ASF licenses this file to you under the Apache - * License, Version 2.0 (the "License"); you may not use this file - * except in compliance with the License. You may obtain a copy of - * the License at http://www.apache.org/licenses/LICENSE-2.0 . - */ - -#ifndef INCLUDED_BASEGFX_RASTER_BPIXELRASTER_HXX -#define INCLUDED_BASEGFX_RASTER_BPIXELRASTER_HXX - -#include <memory> -#include <algorithm> -#include <string.h> -#include <sal/types.h> -#include <basegfx/pixel/bpixel.hxx> -#include <basegfx/basegfxdllapi.h> - -namespace basegfx -{ - class BPixelRaster - { - private: - BPixelRaster(const BPixelRaster&) = delete; - BPixelRaster& operator=(const BPixelRaster&) = delete; - - protected: - sal_uInt32 mnWidth; - sal_uInt32 mnHeight; - sal_uInt32 mnCount; - std::unique_ptr<BPixel[]> mpContent; - - public: - // constructor/destructor - BPixelRaster(sal_uInt32 nWidth, sal_uInt32 nHeight) - : mnWidth(nWidth), - mnHeight(nHeight), - mnCount(nWidth * nHeight), - mpContent(new BPixel[mnCount]) - {} - - // coordinate calcs between X/Y and span - sal_uInt32 getIndexFromXY(sal_uInt32 nX, sal_uInt32 nY) const { return (nX + (nY * mnWidth)); } - - // data access read - sal_uInt32 getWidth() const { return mnWidth; } - sal_uInt32 getHeight() const { return mnHeight; } - - // data access read only - const BPixel& getBPixel(sal_uInt32 nIndex) const - { - assert(nIndex < mnCount && "Access out of range"); - return mpContent[nIndex]; - } - - // data access read/write - BPixel& getBPixel(sal_uInt32 nIndex) - { - assert(nIndex < mnCount && "Access out of range"); - return mpContent[nIndex]; - } - }; -} // end of namespace basegfx - -#endif // INCLUDED_BASEGFX_RASTER_BPIXELRASTER_HXX - -/* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/include/basegfx/raster/bzpixelraster.hxx b/include/basegfx/raster/bzpixelraster.hxx index 9f6900d3b125..a9fc6a123d1e 100644 --- a/include/basegfx/raster/bzpixelraster.hxx +++ b/include/basegfx/raster/bzpixelraster.hxx @@ -20,51 +20,73 @@ #ifndef INCLUDED_BASEGFX_RASTER_BZPIXELRASTER_HXX #define INCLUDED_BASEGFX_RASTER_BZPIXELRASTER_HXX -#include <memory> -#include <basegfx/raster/bpixelraster.hxx> #include <basegfx/basegfxdllapi.h> +#include <basegfx/pixel/bpixel.hxx> +#include <basegfx/raster/bzpixelraster.hxx> #include <osl/diagnose.h> +#include <sal/types.h> +#include <memory> +#include <algorithm> +#include <string.h> namespace basegfx { - class BZPixelRaster : public BPixelRaster + class BZPixelRaster { + private: + BZPixelRaster(const BZPixelRaster&) = delete; + BZPixelRaster& operator=(const BZPixelRaster&) = delete; protected: - // additionally, host a ZBuffer + sal_uInt32 mnWidth; + sal_uInt32 mnHeight; + sal_uInt32 mnCount; + std::unique_ptr<BPixel[]> mpContent; std::unique_ptr<sal_uInt16[]> mpZBuffer; public: // constructor/destructor BZPixelRaster(sal_uInt32 nWidth, sal_uInt32 nHeight) - : BPixelRaster(nWidth, nHeight), + : mnWidth(nWidth), + mnHeight(nHeight), + mnCount(nWidth * nHeight), + mpContent(new BPixel[mnCount]), mpZBuffer(new sal_uInt16[mnCount]) { memset(mpZBuffer.get(), 0, sizeof(sal_uInt16) * mnCount); } + // coordinate calcs between X/Y and span + sal_uInt32 getIndexFromXY(sal_uInt32 nX, sal_uInt32 nY) const { return (nX + (nY * mnWidth)); } + + // data access read + sal_uInt32 getWidth() const { return mnWidth; } + sal_uInt32 getHeight() const { return mnHeight; } + + // data access read only + const BPixel& getBPixel(sal_uInt32 nIndex) const + { + assert(nIndex < mnCount && "Access out of range"); + return mpContent[nIndex]; + } + + // data access read/write + BPixel& getBPixel(sal_uInt32 nIndex) + { + assert(nIndex < mnCount && "Access out of range"); + return mpContent[nIndex]; + } + // data access read only const sal_uInt16& getZ(sal_uInt32 nIndex) const { -#ifdef DBG_UTIL - if(nIndex >= mnCount) - { - OSL_FAIL("getZ: Access out of range (!)"); - return mpZBuffer[0L]; - } -#endif + assert(nIndex < mnCount && "Access out of range"); return mpZBuffer[nIndex]; } // data access read/write sal_uInt16& getZ(sal_uInt32 nIndex) { -#ifdef DBG_UTIL - if(nIndex >= mnCount) - { - OSL_FAIL("getZ: Access out of range (!)"); - return mpZBuffer[0L]; - } -#endif + assert(nIndex < mnCount && "Access out of range"); return mpZBuffer[nIndex]; } }; |