Add Windows CI for msvc 2017 and msys2

Currently the container is based upon a windows 1607 image
and requires a runner with a server/hyper-v 1607 host
to be able to execute.

There are plan on moving the runner to a 1807 later.

uwp: workaround a false positive in certification of orc

It seems that the Windows App Certification Kit searches all files and
binaries for the regex '\<reg\>' (or something like it) and throws
errors if it exists. Supposedly this is for preventing apps from
running REG.EXE

https://blogs.msdn.microsoft.com/appconsult/2017/08/16/how-to-validate-if-your-application-is-compliant-with-the-windows-store-polices-windows-10-and-windows-10-s/

Meson: Do not always generate static library

Since Meson 0.46.0 'default_library' can be set to 'both' when user
wants both shared and static libraries.

Back to development

Release 0.4.29

orc: fix C++ style comment

configure.ac: no more bugzilla

Update news file for upcoming release

Add pkg-config file for orc-test library

Fixes #16

Fix warnings with Cerbero's mingw (gcc 4.7)

orcutils.h:59:38: error: "__STDC_VERSION__" is not defined [-Werror=undef]

meson: Make orcc detectable from other subprojects

If we are a subproject (e.g., gst-build), this can make it possible
to use orcc by other subprojects.

PowerPC: Support ELFv2 ABI

The ELFv2 ABI does not use function descriptors.  This change has been
tested on Power Mac G5, IBM POWER6, and Raptor Talos II, all using the
musl libc on Linux kernel 4.14.

Add support for ppc64le

Fixes https://gitlab.freedesktop.org/gstreamer/orc/issues/6

ci: add minimal ci

orc: Remove dead assignments

All those temporary registry were not used and instead were being
assigned constant values.

meson: Maintain macOS ABI compatibility

Need to use the correct dylib versioning.

meson: add option to disable orc-test library

https://bugzilla.gnome.org/show_bug.cgi?id=736785

meson: rename "enable-backend" option to "orc-backend"

More consistent with the Meson option naming conventions.

meson: add options to disable benchmarks, examples, tools

Like we have in other modules.

meson: make "tests" option an auto feature option

Consistent with the rest of GStreamer and gst-build.

meson: rename 'disable_gtkdoc' option to 'gtk_doc' and make it an auto/feature option

meson: rename option disable_tests=false -> tests=true

Bring it in line with convention.

meson: Bump required version to 0.46

meson: pass the right header when testing for posix_memalign

has_function() needs to know the header to correctly detect the function in
all cases.

This fixes the build under mingw-w64 where posix_memalign was wrongfully being
detected as available.

See this glib bug for a similar fix:
    https://bugzilla.gnome.org/show_bug.cgi?id=795876

https://bugzilla.gnome.org/show_bug.cgi?id=796309

Fix MSVC compiler warning

The isnan() macro was redefined to _isnan() on MSVC, but it's
been in math.h since vs2013, so it should be safe to rely on
it unconditionally nowadays, and we have no intention of
supporting ancient vs versions anyway.

orc: fix static build on windows

Orc users mustn't use __declspec(dllimport) when
linking against a static liborc.

orctest: Allow enough room for sprintf()

We were writing more than 300 characters in some cases. Thanks gcc
8 for pointing that out :)

meson: use add_project_arguments() instead of add_global_arguments()

The latter won't play nice when being used as a subproject.

orc: update some generated files

orcfunctions.h: include orcutils.h for ORC_API define

Fixes build for users that just include orcfunctions.h
directly instead of orc/orc.h, like libs in gst-plugins-base.

orc: remove empty and unused orcrules-x86.c

orc: move unusable orc_code_chunk_free() declaration to internal header

Nothing in any public header ever returned or used an OrcCodeChunk.

autotools: control symbol visibility via -fvisibility

All symbols are hidden by default unless exported
explicitly via our public API decorators.

meson: fix symbol export with MSVC and use -fvisibility elsewhere

Use newly-added ORC_API decorators to export symbols explicitly.

Get rid of orc.map file and the ORC_EXPORT stuff that never did
anything anyway (define was unused).

orc: decorate public symbols with ORC_API

We'll use that later to control symbol visibility and
exported symbols.

orc-test: decorate public symbols with ORC_TEST_API

We'll use that later to control symbol visibility and
exported symbols.

orcc: add --decorator command line argument

So we can add things like FOO_API if the functions are
supposed to be public, for example.

meson: use -Bsymbolic-functions when linking if available

cc.has_linker_argument() requires Meson 0.46, so guard
against older Meson versions for now.

orc: make more functions static

All of these are only used internally and are
not in any header files, but were still exported
by accident because they start with orc_*.

gtk-doc: get rid of templates

gtkdoc-mktmpl was removed in recent gtk-doc versions.

Update gtk-doc.make to 1.27, but modify to generate
xml/version.entities (i.e. add copy in right dir).

All a bit cargo-culted, but seems to work.

orcmips: only enable if the DSPr2 ASE is present

The mips target uses various instructions that a part of the MIPS DSPr2
ASE which is not present on all processors. The easiest solution is to
gate the target on the presense of DSPr2 at runtime.

Unfortunately there isn't a "nice" way to detect if DSPr2 is present, so
resort to parsing /proc/cpuinfo to get the information.

https://bugzilla.gnome.org/show_bug.cgi?id=760834

orcutils: refactor /proc/cpuinfo reading code from orccpu-arm.c

This code will later be used by mips target, so it makes sense
to use a common function instead of duplicating the code.

https://bugzilla.gnome.org/show_bug.cgi?id=760834

configure: don't check for functions that aren't used anywhere

And remove some unneeded includes.

meson: fix mips cpu_family test

The cpu family "mipsel" does not exist in meson so the current test to
enable the HAVE_MIPSEL macro is always false. The real cpu family is
"mips" but this also matches big endian platforms so also check that the
endian is little.

https://bugzilla.gnome.org/attachment.cgi?id=370595

orccpu-arm: Allow 'cpuinfo' fallback on non-android

Reading the cpu flags from cpuinfo is also useful on non-android
ARM platforms which might have mechanisms that prevent /proc/*/auxv
from being read.

meson: Always build both shared and static libraries

This reuses objects from the shared library for the static library, so
it should not affect the build time at all.

orc_dep and orc_test_dep will still contain the library selected by
--default-library

Build liborc-test also as a properly versioned library with meson

Add meson configuration for disabling tests and gtk-doc docs

https://bugzilla.gnome.org/show_bug.cgi?id=790626

Allow building shared/static library based on configuration

Before only a shared library was possible

Back to development

Release 0.4.28

orcutils: Use compiler offsetof() when available

The way we were calculating the offset of a structure entry was
undefined behaviour.

Instead, use 'offsetof()' (defined in C89) when available

orccpu-x86: Fix undefined behaviour compilation issue

runtime error: left shift of 1 by 31 places cannot be represented in type 'int'

configure.ac: Print configuration after configuration

Makes it easier to spot/review detected flags

orc: Add a configure option to disable tests

For targets where it doesn't make sense.

Note that this will disable the build of orc-bugreport since it
depends on the the orc_test helper library

tesuite: Fix some leaks

orcprogram-c.c: Don't cast constant

Clang compilers would complain with constant-conversion error (trying
to fit a 'int' into another type)

This was introduced to fix a warning in gcc ... which doesn't happen

orc: Cast maximum constants

Avoids constant-conversion warnings as such:

/src/orc/orc/orcemulateopcodes.c:158:13: error: implicit conversion from 'int' to 'orc_int8' (aka 'signed char') changes value from 255 to -1 [-Werror,-Wconstant-conversion]
    var34 = ORC_CLAMP_UB((orc_uint8)var32 + (orc_uint8)var33);
          ~ ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/src/orc/orc/orcemulateopcodes.c:29:48: note: expanded from macro 'ORC_CLAMP_UB'
                                               ^~~~~~~~~~
/src/orc/orc/orcemulateopcodes.c:18:20: note: expanded from macro 'ORC_UB_MAX'
                   ^~~
/src/orc/orc/orcemulateopcodes.c:12:55: note: expanded from macro 'ORC_CLAMP'
                                                      ^

Dist meson files with autotools make dist

Back to development

Release 0.4.27

orcc: make coverity happy

Use typedefed enum so static analysis tools like coverity can know
we covered all possibilities in switch/case statements. CID 1413336.

orc: preserve non volatile sse registers

required for MSVC
https://msdn.microsoft.com/en-us/library/9z1stfyw.aspx

orc already marks these register in compiler->save_regs,
but the actual saving and restoring was not done

https://bugzilla.gnome.org/show_bug.cgi?id=783931

meson: Only use --version-script if not using MSVC

meson: use find_library() on compiler object

The old stand-alone find_library() was deprecated.

meson: add meson build

Includes contributions from Nirbheek Chauhan <nirbheek@centricular.com>

orcx86: Don't hard-code register size to zero in orc_x86_emit_*() functions

Instead use the size passed as argument. Fixes segmentation fault on QNX.

https://bugzilla.gnome.org/show_bug.cgi?id=779319

orcx86: use HAVE_AMD64 and HAVE_I386 defines everywhere

Some places were checking for __amd64__ and __i386__ explicitly,
but MSVC doesn't define __amd64__ in 64-bit mode, so use the
define from config.h instead. Fixes incorrect asm generation
on 64-bit Windows

Back to development

Release 0.4.26

pkgconfig: add orc-test path to the -uninstalled .pc file

Expose orc-test library in the uninstalled pkgconfig file.

use 64bit arithmetic to increment the stride

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=770295

orcx86: Fix generation of ModR/M / SIB bytes for the EBP, R12, R13 registers

See http://wiki.osdev.org/X86-64_Instruction_Encoding#ModR.2FM

ESP and R12 are to be handled exactly the same, that is the addressing is
done with SIB instead of directly.

EBP and R13 have a special case for Mod 00, where they're expecting RIP/EIP
plus 32 bit displacement. If we handle them like all other registers, we would
instead of addressing offset 0, address something relative to the instruction
pointer and make part of the following instruction the 32 bit displacement.

Instead of using Mod 00 for offset 0 like for all other registers, we use Mod
01 and a 8 bit displacement of 0.

On Windows this rather low-level bug caused problems at a rather
high-level in the ORC video conversion code, where a movdqu was eating the
following paddw instruction and tried to load data from a rather random memory
location.

For whatever reason this didn't cause any other big problems in other ORC
generated code, and it also only seemed to have caused problems on Windows.

https://bugzilla.gnome.org/show_bug.cgi?id=765713

tests: fix a warning spotted when building in osx

orctest.c:730:19: error: logical not is only applied to the left hand side of this comparison
       if (!float_compare (dest_emul[l-ORC_VAR_D1], dest_exec[l-ORC_VAR_D1], i, j) != 0) {

It seems some sort of typo. Fixing by removing the potentially leftover condition.

Fix test_parse with no executable backend

https://bugzilla.gnome.org/show_bug.cgi?id=765953

Back to development

Release 0.4.25

configure: fix bug report url

orcc: check version of liborc to make sure the right lib is being used

Interesting compilation failures might otherwise ensue.

Add orc_version_string()

uninstalled.pc: add support for non libtool build systems

Currently the .la path is provided which requires to use libtool as
mentioned in the GStreamer manual section-helloworld-compilerun.html.
It is fine as long as the application is built using libtool.

So currently it is not possible to compile a GStreamer application
within gst-uninstalled with CMake or other build system different
than autotools.

This patch allows to do the following in gst-uninstalled env:
gcc test.c -o test $(pkg-config --cflags --libs orc-uninstalled)
Previously it required to prepend libtool --mode=link

https://bugzilla.gnome.org/show_bug.cgi?id=720778

orcprogram-c: check array bound in get_varname_stride()

https://bugzilla.gnome.org/show_bug.cgi?id=759840

Revert "orcprogram-c: avoid running out of bounds of varnames"

This reverts commit 951091788b8496868bf86b2f853f5214659e782a.

This isn't a proper fix and the risk of bringing in a regression is too
high for the small and incomplete improvement.

configure: Only check for Android's liblog on Android targets

https://bugzilla.gnome.org/show_bug.cgi?id=760008

configure: Make -Bsymbolic check work with clang.

Update the -Bsymbolic check with the version glib has. This version
works with clang.

https://bugzilla.gnome.org/show_bug.cgi?id=759713

orcprogram-c: avoid running out of bounds of varnames

These two for loops use varnames[i], since varname has 48 items avoid
running out of bounds by running the for loop up to that limit.

CID 1147004

doc: update from entropywave locations to gstreamer

The entropywave website isn't even up any longer.
Docs have been uploaded to gstreamer website too now.

doc: update generated opcode tables

tools: remove unused test.orc

build: fix detection of AMD Kn host cpu

Not that anyone noticed in all these years, so
the check is probably bogus.

executor: fix load of parameters smaller than 64 bits

Parameters less than 64 bits are passed in just one param register.
Fixes crashes on ldresnearb and friends in emulated code.

compiler: also prefer the backup function when no target

If we don't have a compiler target (like for ppc64), prefer to use the
backup function instead of doing full emulation.

test-limits: improve test without target

When there is no target available, the unit test would fail with a
non-fatal error (because emulation would be done). The non-fatal error
would however not be overwritten by the real error when we exceed the
limits.

First relax the first compilation test to check for FATAL errors. Then
reset the program (and clear any non-fatal errors) before triggering
the next error.

Back to development

Release 0.4.24

parse: increase array size

Increase the size of the array to hold our variable name and use
snprintf to avoid overflows.

program: only reuse constants of the same size and value

When reusing an existing constant, make sure not only the value but also
the size is the same. We then also need to make a name based on the size
and value so that we can store the same value for different sizes.

Fixes problem with the bayer functions that used 255 as a 16 and 8 bit
constant. This was not detected in the unit test because both the backup
and asm functions do the same error, but it could be seen when a new
backup function was checked against a older ORC.

testsuite: fix reading .orc files with win32 line endings on windows

Just read them in binary form with win32 line endings and
parse them to the orc parser as-is now that the parser
can handle win32 line endings.

testsuite: add parsing test case for win32 line endings

Should handle \r\n as well as \n instead of relying on
the file reader to translate it.

orcc: fix reading .orc files with win32 line endings on windows

We used to open the files in text/translation mode, which
makes fread() convert all windows-style line endings (\r\n)
to unix style line endings (\n) under the hood. However, the
file size we get from the seek is apparently still the original
binary size, and then we just get less data because the \r have
been removed. This means we put our zero terminator in the wrong
place, and there might be some junk between the end of the
text data and the zero terminator.

Fix this by reading the file in binary mode (who knows what
else the text mode might mess up or trip over, e.g. unicode
chars in comments or such) and pass it to the orc parser
as-is now that the parser can handle win32 line endings.

Spotted by Ray Molenkamp.