[clang-tidy] do not use else after return

Found with readability-else-after-return

Signed-off-by: Rosen Penev <rosenp@gmail.com>

[clang-tidy] use empty()

Found with readability-container-size-empty

Signed-off-by: Rosen Penev <rosenp@gmail.com>

[clang-tidy] make parameter names match those in function declarations

KaxSemantic.cpp: add missing function implementations for KaxChapterTranslate*

KaxSemantic.h: fix missing function declarations

KaxSemantic.h: remove superfluous #endif

Merge pull request #33 from Matroska-Org/no-divx-fileused

do not write DivX FileUsed elements

fix ChapLanguageIETF being assumed as unique

The EBML path is 0*(\Segment\Chapters\EditionEntry\ChapterAtom\ChapterDisplay\ChapLanguageIETF)

Merge pull request #30 from Matroska-Org/enc-algo-mandatory

fix ContentEncAlgo assumed not to be mandatory

Merge pull request #21 from Matroska-Org/forbid-slices

forbid writing TimeSlice, SliceLaceNumber and TrackAttachmentLink

Merge pull request #48 from Matroska-Org/write-track-scale

Allow writing TrackTimecodeScale

Merge pull request #32 from Matroska-Org/new-aes-elements

add ContentEncAESSettings and AESSettingsCipherMode

Merge pull request #45 from Matroska-Org/blockadd-mapping

add BlockAdditionMapping support

Merge pull request #42 from Matroska-Org/remove-MATROSKA_VERSION

Remove MATROSKA_VERSION 2 conditional compilation

add ContentEncAESSettings and AESSettingsCipherMode

removed MATROSKA_VERSION for KaxBlockAdditionMapping

Allow writing TrackTimecodeScale

TrackTimestampScale/TrackTimecodeScale was allowed until Matroska v3 so it
should be possible to remux files with the value.

add BlockAdditionMapping support

Remove MATROSKA_VERSION 2 conditional compilation

Bump the SO version as it's an ABI breakage.
Bump the library to 1.6.0 as it's an API breakage (for outdated softwares).

Fixes #41

ChangeLog: update for PR 36

Merge pull request #36 from Maturin/master

#35: Fixed linking against libmatroska.dll (lib)

#35: Fixed linking against libmatroska.dll (lib)

fix ContentEncAlgo assumed not to be mandatory

The EBML Path is 1*1(\Segment\Tracks\TrackEntry\ContentEncodings\ContentEncoding\ContentEncryption\ContentEncAlgo)

do not write DivX FileUsed elements

they are deprecated and not well documented

bump version number to 1.5.2

require libebml 1.3.9

update ChangeLog

Merge pull request #28 from evpobr/fix-export-symbols

Fix symbols export

Fix symbols export, closes #50

bump version number to 1.5.1

ChangeLog: update

build system: require libEBML 1.3.8

Merge pull request #27 from Rupan/cmake_bugfix

Fix a typo in the CMake file

Fix a typo in the CMake file

Fix C++ visibility preset

Mark KaxInternalBlock as exported

Classes derived from KaxInternalBlock are exported,  but KaxInternalBlock is not.

Fixes compiler warning.

CMake fixes

KaxBlock: do not attempt to use laced sizes that are clearly invalid

KaxBlock: don't reset potentially unallocated memory

When using SCOPE_PARTIAL_DATA the GetBuffer()/Data pointer is never allocated.
If this exception occurs we must not write on NULL pointed data.

When the memory is allocated it doesn't have any use to reset the memory to 0
since SetValueIsSet(false) is called right before, invalidating the buffer.

bump version number to 1.5.0

update ChangeLog, add release message

require libEBML 1.3.7

libEBML 1.3.7 fixes handling of Unicode code points > U+ffff.

README.md: fix docs regarding specifying custom path for libEBML

remove extra ; at end of classes & namespaces

The various macros creating classes already contain the required ;
after the class. Adding an extra ; after the macro invocation results
in compiler warnings with -Wpedantic.

Namespace definitions don't require trailing ; in the first place
resulting in the same warning.

Merge pull request #19 from schultetwin1/fix_cmake_module_path

Remove explict CMAKE_MODULE_PATH set

forbid writing TimeSlice, SliceLaceNumber and TrackAttachmentLink

Remove explict CMAKE_MODULE_PATH set

Currently, CMAKE_MODULE_PATH is expliclty overridden and set to
"${CMAKE_CURRENT_SOURCE_DIR}/cmake/". Since that directory does not
exist this prevents anyone from writing a FindEbml.cmake file. This
works if you are consuming libeml by installing it on your system.

This breaks the use case of having libebml built as part of your build.
If you have called add_subdirectory(libebml) already, the build should
be able to continue. I'm removing the explict set of CMAKE_MODULE_PATH
to allow this use case.

bump version number to 1.4.9; include release message

ChangeLog: update

ChangeLog: update for change to cmake

cmake: fix requirement name of libebml

libebml's cmake file is called EbmlConfig.cmake.

cmake: fix version number

cmake: fix setting CMAKE_INSTALL_PACKAGEDIR on non-Windows platforms

cmake: move MatroskaConfig.cmake to main directory to match libebml

README.md: update for usage of cmake

ignore the build/ directory

remove autoconf/automake files

Merge remote-tracking branch 'evpobr/feature/cmake' into evpobr-cmake

use the same term to tell there has been modificiations to the original

Add some language to avoid people behaving badly elsewhere

Add the Contributor Covenant code of conduct

bumped version number, added release message

KaxBlockGroup, KaxSimpleBlock: fix writing track number == 128

Add CMake project

bump version number

Bump version number, add release message

update ChangeLog, add release message

Revert "forbid the use of KaxTimeSlice and KaxSliceLaceNumber as they are deprecated since v1"

This reverts commit 3c698a372abe63a60b60309f2bbeaf12d1c15979. It
isn't really required, but it breaks ABI compatibility.

forbid the use of KaxTimeSlice and KaxSliceLaceNumber as they are deprecated since v1

add IETF BPC 47 language elements

update coremake project

generate semantics using the new spectools, using the XML Schema for Matroska

aka ebml_specification.xml

The display name for projection items has changed.

Merge pull request #11 from MaddTheSane/documentationWarning

Quiet warnings in clang when -Wdocumentation is set.

Quiet warnings in clang when -Wdocumentation is set.

add 360°/VR elements

manually edited until the new XML Schema format is processed correctly

README: add a README with basic installation instructions

Bump version number, add release message

add colour-related elements from latest specdata.xml

add the FieldOrder element from the latest specdata.xml

KaxVersion.cpp: always set KaxCodeDate to "Unknown"

This is done in order to enable deterministic builds while still keeping
the library API and ABI compatible. Patch by Ed Schouten <ed@nuxi.nl>.

Bump version number, add release message

build system: require libebml >= 1.3.3 due to fixes for invalid memory access

KaxBlockInternal: check EBML lace sizes against available buffer space

Bump version number, add release message

Merge pull request #7 from DanWin/master

Some optimizations

Fix memory leak

Some optimizations

build: libmatroska needs to link against libebml

The library uses symbols from libebml, and so should link to it.

Merge pull request #5 from phonohawk/fix-libmatroska-linkage-failure

Fix linkage failure of libmatroska due to missing -lebml flag

Fix linkage failure of libmatroska due to missing -lebml flag

Without this change the following linkage error occurs:

    /bin/sh ./libtool  --tag=CXX   --mode=link g++ -Wall -Wextra \
      -Wno-unknown-pragmas -Wshadow -I/usr/pkg/include   -pipe -O2 \
      -I/usr/pkg/include -version-info 6:0:0 -no-undefined -L/usr/pkg/lib \
      -o libmatroska.la -rpath /usr/pkg/lib src/FileKax.lo \
      src/KaxAttached.lo src/KaxAttachments.lo src/KaxBlock.lo \
      src/KaxBlockData.lo src/KaxCluster.lo src/KaxContexts.lo \
      src/KaxCues.lo src/KaxCuesData.lo src/KaxInfoData.lo \
      src/KaxSeekHead.lo src/KaxSegment.lo src/KaxSemantic.lo \
      src/KaxTracks.lo src/KaxVersion.lo
    Undefined symbols:
      "libebml::EbmlUnicodeString::EbmlUnicodeString()", referenced from:
          libmatroska::KaxSegmentFilename::KaxSegmentFilename()in KaxSemantic.o
          libmatroska::KaxPrevFilename::KaxPrevFilename()in KaxSemantic.o
          libmatroska::KaxNextFilename::KaxNextFilename()in KaxSemantic.o
          ...

ChangeLog: add release message

build system: autogen.sh not needed; »autoreconf -fi« suffices

ChangeLog: mention the pkg-config file

ChangeLog: update

build system: remove old hand-crafted make files

check_ids: move to other directory

build system: add autoconf/automake-based system

cosmetics: re-indent KaxBlock::ReadData() after wrapping in try/catch block

KaxBlock::ReadData(): use safe/checked memory access

Unchecked reading from memory locations works for cases in which all the
data is valid, but if it isn't then this leads to invalid memory access
and segmentation faults.

See https://trac.bunkus.org/ticket/1096

cosmetics: unify indentation