Corrected various spelling mistakes in comments, documentation, and
variable names across the project. Changes include:
- Documentation files (CONTRIBUTING.md, README.md, etc.)
- Source code comments in C/C++ files
- Function parameter names and descriptions
No functional changes were made.
Added additional number validitiy check (to newly created parse_number
function). Mainly to cover inputs like '1something' where it should
not be parsed correctly as being 1.
This is a hack to check whether given invalid_argument message belongs
to stoi/stod. It is usually used to catch non-numeric user input where
a number was expected.
moved compat functions from config_*h, namely:
- localtime_s/gmtime_s
- strerror_s
- mkdir Windows compat
- removed str[n]cmp from compat/misc (already in compat/strings,h,
which is the better location)
Do not use config*.h in export.c (motivation for the above changes).
Encounter using standard strerror_s() instead of home-grown
replacement. MSW already include C11-compatible strerror_s()
implementation (although it doesn't advertise __STDC_LIB_EXT1__).
XSI version of strerror_r() is mostly compatible (except of argument
order).
Using standard functions should be more convenient than custom workarounds
(even if there is needed a compat macro).
While on x86_64 is libsvtav1 still slightly ahead, on the M1 mac, the
AOM AV1 performs significantly better (2x faster).
The above holds for native builds, it actually looks like the x86_64
build running SVT AV1 on M1 mac doesn't run correctly at all - it
produces just blank picture (green as is zeroed YCbCr buffer).
- moved macros to utils/macros.h (not config_common.h that is not going
to be included) and include in config_common.h the macros.h file
instead (later it should be removed)
- avoid dependency of color_out.o on host.o if build outside UG (easiest
for now)
- compile tools with '-g' (obviously for better debuggability)
In response to Coverity CID 383367 - it is perhaps false positive but
there has been another small bug - the step should be length of to, not
from (which may be longer).
+ written a test for it
Set signal as progressive (not-PsF) by default - previously the value
was not touched which may be a bit misleading when DeckLink outputs
actually interlaced signal.
Issue warning that the option is set and keep user opt out from setting
this value.
