    /* -*- 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 .
     */

    #include <ByteGrabber.hxx>
    #include <sal/log.hxx>
    #include <com/sun/star/io/XSeekable.hpp>
    #include <com/sun/star/io/XInputStream.hpp>

    using namespace ::com::sun::star;

    #if OSL_DEBUG_LEVEL > 0
    #define THROW_WHERE SAL_WHERE
    #else
    #define THROW_WHERE ""
    #endif

    /** ByteGrabber implements the >> operators on an XOutputStream. This is
     *  potentially quite slow and may need to be optimised
     */

    ByteGrabber::ByteGrabber(uno::Reference  < io::XInputStream > const & xIstream)
    : xStream(xIstream)
    , xSeek (xIstream, uno::UNO_QUERY )
    , aSequence ( 4 )
    {
        pSequence = aSequence.getArray();
    }

    ByteGrabber::~ByteGrabber()
    {
    }

    void ByteGrabber::setInputStream (const uno::Reference < io::XInputStream >& xNewStream)
    {
        ::osl::MutexGuard aGuard( m_aMutex );
        xStream = xNewStream;
        xSeek.set(xNewStream, uno::UNO_QUERY);
    }

    // XInputStream chained
    sal_Int32 ByteGrabber::readBytes( uno::Sequence< sal_Int8 >& aData,
                                            sal_Int32 nBytesToRead )
    {
        ::osl::MutexGuard aGuard( m_aMutex );
        return xStream->readBytes(aData, nBytesToRead );
    }

    // XSeekable chained...
    void ByteGrabber::seek( sal_Int64 location )
    {
        ::osl::MutexGuard aGuard( m_aMutex );
        if (!xSeek.is() )
            throw io::IOException(THROW_WHERE );

        sal_Int64 nLen = xSeek->getLength();
        if ( location < 0 || location > nLen )
            throw lang::IllegalArgumentException(THROW_WHERE, uno::Reference< uno::XInterface >(), 1 );
        if (location > nLen )
            location = nLen;
        xSeek->seek( location );
    }

    sal_Int64 ByteGrabber::getPosition(  )
    {
        ::osl::MutexGuard aGuard( m_aMutex );
        if (!xSeek.is() )
            throw io::IOException(THROW_WHERE );

        return xSeek->getPosition();
    }

    sal_Int64 ByteGrabber::getLength(  )
    {
        ::osl::MutexGuard aGuard( m_aMutex );
        if (!xSeek.is() )
            throw io::IOException(THROW_WHERE );

        return xSeek->getLength();
    }

    sal_uInt16 ByteGrabber::ReadUInt16()
    {
        ::osl::MutexGuard aGuard( m_aMutex );

        if (xStream->readBytes(aSequence, 2) != 2)
            return 0;

        pSequence = aSequence.getConstArray();
        return static_cast <sal_uInt16>
               ( (pSequence[0] & 0xFF)
              | (pSequence[1] & 0xFF) << 8);
    }

    sal_uInt32 ByteGrabber::ReadUInt32()
    {
        ::osl::MutexGuard aGuard( m_aMutex );

        if (xStream->readBytes(aSequence, 4) != 4)
            return 0;

        pSequence = aSequence.getConstArray();
        return static_cast < sal_uInt32 >
                ( (pSequence[0] & 0xFF)
              | ( pSequence[1] & 0xFF ) << 8
              | ( pSequence[2] & 0xFF ) << 16
              | ( pSequence[3] & 0xFF ) << 24 );
    }

    /* vim:set shiftwidth=4 softtabstop=4 expandtab: */

↑ V547 Expression 'location > nLen' is always false.