Fix identical inner if condition rrd_graph_helper

- Fixes the following Cppcheck warning:
  [src/rrd_graph_helper.c:972] (warning) Identical inner 'if' condition
  is always true
  (outer condition is '!vname' and inner condition is '!vname').
  [identicalInnerCondition]

Indent rrd_graph_helper.c

- indent src/rrd_graph_helper.c using GNU indent 2.2.12

Fix possible memory leak by buffer.file

Close the opened buffer.file in rrd_xport.c before return -1,
if the data processing has failed

- Fixes the following Cppcheck error:
  [rrdtool-1.x/src/rrd_xport.c:435] (error) Memory leak:
  buffer.file [memleak]

Include asprintf.h in rrd_info.c for MSVC builds

- Fixes the following MSVC level 3 compiler warning:
  rrd_info.c(21): warning C4013: 'vasprintf' undefined;
  assuming extern returning int
- This is a follow-up to PR #828 and commit b1bcbca

Add rrdcgi.vcxproj and update Makefile.am (MSVC)

So far, rrdcgi.exe has been built using nmake. Add rrdcgi.vcxproj
as an additional Visual Studio project and add it to rrd.sln

- Update Makefile.am:
  Add win32/rrdcgi.vcxproj and win32/Makefile_vcpkg.msc to EXTRA_DIST
  Remove win32/rrdtool.sln and win32/rrdupdate.sln, because all
  projects are part of rrd.sln
- git rm win32/rrdtool.sln win32/rrdupdate.sln

Update MSVC rrd.sln and .vcxproj files

- rrd.sln:
  Do not build rrdtool and rrdupdate in case of configurations
  DebugDLL|x64 and ReleaseDLL|x64
- rrdtool.vcxproj:
  Move <AdditionalIncludeDirectories> before <PreprocessorDefinitions>
  like in librrd-8.vcxproj, rrdupdate.vcxproj and the other occurrences
  in rrdtool.vcxproj
- rrdupdate.vcxproj:
  Change GenerateDebugInformation for Release configurations from
  true to false

Fix UB when calculating median of all-NaN values

The current code contains undefined behavior where all-NaN values
are passed to median. In that case we end up with final_elements==0
in the following branch:

    else {
        rpnstack->s[++stptr] =
            0.5 * (element_ptr[final_elements / 2] +
                   element_ptr[final_elements / 2 - 1]);
    }

and so we use 0 and -1 as element_ptr array indexes. The
latter is ill-formed and leads to a crash in my case. Move the
check which accounts for the last NaN earlier, so we could
push NaN and finish right away.

Add platform x64 to MSVC .sln and .vcxproj files

So far, MSVC x64 builds have been created using nmake. This adds x64 to
the .sln and .vcxproj files and x64 can be chosen within Visual Studio.

- Update the created x64 platform configurations with the matching
  include and lib directories
- Remove unnecessary and incorrect <AdditionalIncludeDirectories>
  from rrdtool.vcxproj and rrdupdate.vcxproj (one ../ was too much)

Bump version to librrd-8.dll for MSVC builds

The current version in the filename of the library is 8, e.g.
librrd.so.8 or librrd-8.dll (MinGW-w64 builds), which is based on
LIBVERS from configure.ac. The version of the dll for MSVC builds is
not derived from LIBVERS and has not been updated yet.

- Substitute occurrences of librrd-4 with librrd-8 using:
  git grep -l 'librrd-4' | xargs sed -b -i 's/librrd-4/librrd-8/g'
- rename librrd-4* files to librrd-8* using:
  find . -name 'librrd-4*' \
  -exec bash -c 'file={}; git mv $file ${file/librrd-4/librrd-8}' \;

Simplify first_weekday under Windows

Use LOCALE_IFIRSTDAYOFWEEK | LOCALE_RETURN_NUMBER. In this case,
the function GetLocaleInfoEx() retrieves the value as a number instead
of a string.
According to: GetLocaleInfoEx function (winnls.h) | Microsoft Docs

sscanf() is not required any more to convert a char array to an int

Implement first_weekday under Windows

- Use GetLocaleInfoEx() to obtain the first weekday

Indent src/rrd_rpncalc.c

- indent src/rrd_rpncalc.c using GNU indent 2.2.12
  before further changes to the file
- Fixes: indent: src/rrd_rpncalc.c:1007:
  Warning:old style assignment ambiguity in "=-". Assuming "= -"

Add vcpkg support for MSVC builds

- This allows building against newer and up-to-date versions of
  required libraries. Vcpkg is a library manager and can be found here:
  https://github.com/microsoft/vcpkg
- Libraries are provided for 32-bit and 64-bit.
  Current versions of libraries are e.g.:
  cairo 1.16.0, expat 2.2.6, fontconfig 2.12.4, freetype 2.9.1,
  gettext 0.19, glib 2.52.3, libpng 1.6.37, pango 1.40.11, pcre 8.41,
  libxml2 2.9.9 and zlib 1.2.11
  Furthermore these libraries from vcpkg are regularly updated.
- Added information to WIN32-BUILD-TIPS.txt and win32/README concerning
  vcpkg
- Added win32/Makefile_vcpkg.msc for building using nmake

Set fallback first_weekday to 0 (fix #1012)

- Set first_weekday to 0 (Sunday), when HAVE__NL_TIME_WEEK_1STDAY
  is not defined
- Fixes: https://github.com/oetiker/rrdtool-1.x/issues/1012

Link to section "HOW TO MEASURE"

Add rrdpython.pod and rrdruby.pod to dist

Adding Rounding Function

Update doc/rrdruby.pod

- Fixes: https://github.com/oetiker/rrdtool-1.x/issues/885
  Typo: --interlace -> --interlaced
- Change encoding of file from ANSI to UTF-8
  add to pod file: =encoding utf8
  Fixes podchecker error:
  *** ERROR: Non-ASCII character seen before =encoding in 'Loïs'.
  Assuming CP1252 at line 84 in file doc/rrdruby.pod
  doc/rrdruby.pod has 1 pod syntax error.

Fix --use-nan-for-all-missing-data

- Fixes: https://github.com/oetiker/rrdtool-1.x/issues/918

Indent rrd_graph.c

- indent src/rrd_graph.c using GNU indent 2.2.12

Add --showtime to xport help output

- Fixes: https://github.com/oetiker/rrdtool-1.x/issues/911

Indent rrd_tool.c

- indent src/rrd_tool.c using GNU indent 2.2.12

Fix failed to expand Py_UNUSED (rrdtoolmodule.c)

- Fix: failed to expand 'Py_UNUSED', Invalid ## usage when expanding
  'Py_UNUSED'.
- Fixes: https://github.com/oetiker/rrdtool-1.x/issues/903

Update rrdtool-release script

- Update version numbers for MSVC builds too. These are defined in
  win32/rrd_config.h
- Update Copyright info and years in win32/*.rc files
- Use a variable for the current year
- Make script compatible with rrdtool major versions > 1
  (use \d instead of 1)
- Minor updates to the script considering recommendations from
  ShellCheck, e.g.:
  Use $(...) notation instead of legacy backticked `...`.

Restore Copyright Years in src/*.c src/*.h

- Restore the Copyright Years, so that rrdtool-release will update the
  years in the future again as intended.
  The years are not present any more since the following commit:
  https://github.com/oetiker/rrdtool-1.x/commit/1964f6f
- rrdtool-release expects a 4 digit year for the substitution:
  s/Copyright.+?Oetiker.+\d{4}

Fix missing double values in rrdinfo.

The double values of rrd info were never properly converted to the
python dictionary causing some keys (e.g. xff) to be missing.

numeric version for perl

released 1.7.2

Define _POSIX_THREAD_SAFE_FUNCTIONS (MinGW-w64)

- time.h of MinGW-w64 requires _POSIX_THREAD_SAFE_FUNCTIONS to be
  defined in order to provide ctime_r, gmtime_r or localtime_r.
- In newer versions of winpthreads (e.g. git-7.0.0.5447.a2d94c81),
  _POSIX_THREAD_SAFE_FUNCTIONS is not defined any more in
  pthread_unistd.h. Hence define it here directly.
- This is a follow-up to commit:
  https://github.com/oetiker/rrdtool-1.x/commit/603de9a

Silence unused variable and parameter warnings

- Cast unused variables and parameters to void, do nothing
- Fixes the following compiler warnings on Windows:
rrd_open.c:
  757:9: warning: unused parameter 'writelock' [-Wunused-parameter]
    int writelock)
  810:24: warning: unused variable 'rrd_simple_file'
    [-Wunused-variable] rrd_simple_file_t *rrd_simple_file;
  807:17: warning: unused parameter 'rrd_file' [-Wunused-parameter]
    rrd_file_t *rrd_file,
  808:12: warning: unused parameter 'rrd' [-Wunused-parameter]
    rrd_t *rrd)
rrd_client.c:
  746:39: warning: unused parameter 'client' [-Wunused-parameter]
    static int connect_unix(rrd_client_t *client, const char *path)
    /* {{{ */
  746:59: warning: unused parameter 'path' [-Wunused-parameter]
    static int connect_unix(rrd_client_t *client, const char *path)
    /* {{{ */

Indent src/rrd_client.c

- indent src/rrd_client.c before any further changes using
  GNU indent 2.2.12

Fix compiler warning: label defined but not used

- The label 'no_lseek_necessary' is only relevant,
  if HAVE_POSIX_FALLOCATE is defined.
- Fixes the following compiler warning on systems, where
  HAVE_POSIX_FALLOCATE is not defined:
  rrd_open.c:387:3: warning: label 'no_lseek_necessary' defined but not
  used [-Wunused-label]

Indent rrd_create.c

- Add typename GList to .indent.pro
- indent rrd_create.c using GNU indent 2.2.12

Avoid MinGW-w64 strndup compiler warning

- Fixes the following MinGW-w64 GCC compiler warning:
  rrd_create.c:287:14: warning: declaration of 'strndup' shadows a
  built-in function [-Wshadow] static char *strndup(
- MinGW-w64: strndup is not detected by configure, therefore
  HAVE_STRNDUP is not defined.
  Use strndup_() here instead of strndup() to avoid any collisions.

Consider flags for CreateFileA(), Windows

- Implement dwDesiredAccess and dwCreationDisposition based on
  the flags O_RDONLY, O_RDWR, O_CREAT | O_TRUNC and O_EXCL
- This updates how a file handle is created using CreateFileA()
  under Windows, considering the flags. Use e.g.
  DesiredAccess = GENERIC_READ | GENERIC_WRITE
  dwCreationDisposition = OPEN_EXISTING
- CreateFileA(): Check for INVALID_HANDLE_VALUE and add output of error
  messages using GetLastError() and FormatMessage()
- The error message provides correct output now, which error occurs.
  e.g.: rrdtool.exe info not_existing_file.rrd
    ERROR: opening 'not_existing_file.rrd':
    The system cannot find the file specified.
  Previously, the following error occurred later in the code:
    ERROR: short read while reading header rrd->stat_head
  And also empty files were left behind:
  e.g. rrdtool.exe resize not_existing_file.rrd 0 GROW 5200
- Use <CharacterSet>MultiByte</CharacterSet> consistently
  in .vcxproj files. There were 4 inadvertent occurrences of
  <CharacterSet>Unicode</CharacterSet> in librrd-4.vcxproj.
  Using MultiByte or NotSet instead of Unicode is required for
  printing "(LPTSTR) lpMsgBuf" from FormatMessage() using %s, to avoid
  unnecessary wide characters.
- This commit is an update to a9671a7

Add missing #define and indent rrd_open.c

- Add missing "#define __rrd_read(dst, dst_t, cnt) { \" after
  #elif defined(HAVE_LIBRADOS) && !defined(HAVE_MMAP)
- This fixes:
    indent: src/rrd_open.c:77: Error:Unmatched 'else'
    indent: src/rrd_open.c:79: Error:Stmt nesting error.
    and errors in case of defined(HAVE_LIBRADOS) && !defined(HAVE_MMAP)
- indent src/rrd_open.c using GNU indent 2.2.12
- Replace all remaining tabs by four spaces

Fix GCC string truncation warnings

- Fixes the following GCC compiler warnings:
  rrd_fetch.c:338:9: warning: 'strncpy' output may be truncated copying
  19 bytes from a string of length 19 [-Wstringop-truncation]
    strncpy((*ds_namv)[i], rrd.ds_def[i].ds_nam, DS_NAM_SIZE - 1);
  rrd_tune.c:284:13: warning: 'strncpy' output may be truncated copying
  19 bytes from a string of length 19 [-Wstringop-truncation]
    strncpy(rrd.ds_def[ds].ds_nam, ds_new, DS_NAM_SIZE - 1);
  rrd_graph.c:2282:17: warning: 'strncpy' output may be truncated
    copying 204 bytes from a string of length 204
    [-Wstringop-truncation] strncpy(im->gdes[i].legend, saved_legend,
    sizeof im->gdes[0].legend - 1);

Avoid pragma comment warning under MinGW

- Removes the following MinGW compiler warning:
  rrd_client.c:32: warning: ignoring #pragma comment
  [-Wunknown-pragmas] #pragma comment(lib, "ws2_32.lib")
- This #pragma comment is only relevant for MSVC

Update .gitignore files

Cast lua_tostring return value to char *

- Fixes the following gcc compiler warning:
  rrdlua.c:69:21: warning: assignment discards ‘const’ qualifier from
  pointer target type [-Wdiscarded-qualifiers]
  if (!(argv[i] = lua_tostring (L, i))) {

Update test tune1

- Compare dump of the updated rrd file with tune1-testorg.dump,
  which is available in tests

Cast unsigned long values to long for labs()

- Fixes the following clang compiler warning:
  rrd_fetch.c:391:29: warning: taking the absolute value of unsigned
  type 'unsigned long' has no effect [-Wabsolute-value]
    tmp_step_diff = labs(*step - (rrd.stat_head->pdp_step

Use fabs() for floating point, labs() for long

- Fixes the following clang compiler warnings:
  rrd_rpncalc.c:999:11: warning: using integer absolute value function
  'abs' when argument is of floating point type [-Wabsolute-value]
    if (abs(percentile) > 100) {
  rrd_rpncalc.c:1103:24: warning: using integer absolute value function
  'abs' when argument is of floating point type [-Wabsolute-value]
    int idx = round(abs(idxf));
  rrd_hw.c:519:36: warning: absolute value function 'abs' given an
  argument of type 'long' but has parameter of type 'int' which may
  cause truncation of value [-Wabsolute-value]
    new_val = ((unsigned long) abs(val)) % mod;

Update strftime.c and use it under MinGW

- Add %F and %T to strftime.c
  These format codes are commonly used, part of C99 and used in the
  test vformatter1
- rrd_graph.c: Include local strftime.h and use strftime_ from
  strftime.c in case of MinGW or MinGW-w64 builds.
- This allows test vformatter1 to pass under MSYS2 (MinGW-w64)

Remove strftime.c and strftime.h from MSVC builds

- Recent versions of MSVC (VS2015 and newer) properly support strftime
  including formatting codes like %F, %T or %V (ISO 8601)

fix segfault on non-existent RRD file when using rrdcached

fix segfault on non-existent RRD file when using rrdcached + rrdtool xport
(like 814ca69a3329ccc88040ed184a413e4e5adf604c does for rrdtool graph)

Update test list1

- Avoid double slashes at beginning of filename paths.
  Leave "/" in CACHED_DIR (only sed ^$BASEDIR), then it is not
  necessary to add "/" in function do_list_tests() several times
  again and "$1" can be used instead of "/$1" there.
- This allows passing of test list1 under MSYS2 (Windows), where paths
  starting (unnecessarily) with // are not working. Exit test list1
  under Windows (MSYS2) before "Starting rrdcached..."

Update functions

Update further URLs from http to https

- Command used to replace http with https:
  git grep -l 'http://tobi.oetiker.ch' -- './*' ':(exclude)CHANGES' | \
  xargs sed -i 's|http://tobi.oetiker.ch|https://tobi.oetiker.ch|g'
- This is an addition to commit: 267369f

Let two more tests pass under MSYS2 (Windows)

- Updates the following tests:
  create-with-source-4, xport1
- Avoid Windows drive letter in the path of DEF:vname=rrdfile
  under MSYS or MSYS2. Use only the filename of the rrd and not
  the full path if environmental variable MSYSTEM is set.

Update URL from http to https

- Command used to replace http with https:
  git grep -l 'http://oss.oetiker.ch' -- './*' ':(exclude)CHANGES' | \
  xargs sed -b -i 's|http://oss.oetiker.ch|https://oss.oetiker.ch|g'
- Fixes:
  https://github.com/oetiker/rrdtool-1.x/issues/934

Update bugs address

- Replace:
  http://oss.oetiker.ch/rrdtool-trac/newticket
  with
  https://github.com/oetiker/rrdtool-1.x/issues

Fix docs on rrdxport

Update xport tests

Fix bounds in rrd_xport

rrdxport needs to provide enough data points in its output for the
receiving client to be able to reproduce the graphs generated by rrdgraph.

- The start time pointer needs to be at the start of the interval that
  contain the query start time. For the function that retrieves the data.
  On output the returned start timestamp is > than query timestamp.
- SV export was missing the end boundary

Fix %s/%S unit autoscaling in gprint for json/xml

Add typenames to .indent.pro, add to EXTRA_DIST

- Several typenames have been added in the meantime using typedef.
  Add them to .indent.pro
- Add .indent.pro file to EXTRA_DIST, so that it is available in the
  tarball
- re-indent src/rrd_xport.c with updated .indent.pro

Indent src/rrd_xport.c, tests/Makefile.am

- Indent src/rrd_xport.c using GNU indent 2.2.12
- tests/Makefile.am:
  Use tab in lines 8 and 22, like in the other lines of Makefile.am

Allow nan and -nan in test pdp-calc1

Added PDP calculation tests

Fixed bug in bug fix. Has to be tested now.

Forgot to increase proc_pdp_cnt after finishing most left open PDP.
Added check for invalid values before calculation, just to be sure.

Optimized PDP calculation when multiple PDPs are closed by an update and the previous one was open

Update etc/Makefile.am

- Add rrdcached-init-lsb to EXTRA_DIST
- Trim trailing space after backslash. Fixes:
  etc/Makefile.am:1: warning: whitespace following trailing backslash

Update snprintf.m4, add _HW_FUNC_XPRINTF_REPLACE

- Add _HW_FUNC_XPRINTF_REPLACE again to snprintf.m4, in a modified way.
  Use AC_LIBOBJ([rrd_snprintf]) instead of original AC_LIBOBJ([snprintf])
- Fixes the following configure messages under MinGW-w64:
  ./configure: line 14694: _HW_FUNC_XPRINTF_REPLACE: command not found
  ./configure: line 14752: _HW_FUNC_XPRINTF_REPLACE: command not found

Delete rrd_config.h.in

Remove obsolete rrd_config.h.in file in the base directory.
The current file is src/rrd_config.h.in

Update year in COPYRIGHT and win32/*.rc files

Delete name.inc

Fixes: https://github.com/oetiker/rrdtool-1.x/issues/968

add notes on update

Define mkdir() macro for Windows builds

- Windows: #define mkdir(A, B) mkdir(A)
  mkdir() has only got one argument under Windows

Call getuid() only if HAVE_GETUID is defined

Update FSF address and fix UTF8

- Update FSF address in COPYRIGHT
  old: 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA
  new: 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
- Update LICENSE file according to the current version from:
  https://www.gnu.org/licenses/gpl-2.0.txt
- Update bindings/python/COPYING according to the current version from:
  https://www.gnu.org/licenses/old-licenses/lgpl-2.1.txt
- Change encoding of CONTRIBUTORS from ISO 8859-1 to UTF-8
- Fixes rpmlint warnings and errors:
  W: file-not-utf8 CONTRIBUTORS
  E: incorrect-fsf-address COPYRIGHT
  E: incorrect-fsf-address LICENSE
  E: incorrect-fsf-address bindings/python/COPYING

add missing testfile

Typo

Thank lintian

add ghostscript to build env

add missing cd ..

consistant numbering for windows

generate NUMVERS from VERSION file

bump version

add releasee tests to travis and bump version number for rerelease with fixed makefiles

Fixed 'No rule to make target 'etc/rrdcached.socket'

Signed-off-by: Jaroslav Škarvada <jskarvad@redhat.com>

properly add etc files via EXTRA_DIST - fixes #956 some more

add missing etc/rrdcached-default-lsb.in to tarball - fixes #956

sync 1.7.1 release updates

update library version since new features were added

fix log

Update version to 1.7.1 for MSVC builds

ready for 1.7.1 release

Remove librrd-4.vcxproj.user

- Remove file from git source control, because
  .vcxproj.user files typically contain user-specific settings
  or overrides

Update issue templates

Fix warnings: different const qualifiers (MSVC)

- Cast affected variables from 'const char **' to 'char **'
- Fixes the following warning:
  C4090: 'function': different 'const' qualifiers
  in: rrd_create.c(248), rrd_modify.c(1312),
  rrd_modify.c(1329), rrd_modify.c(1457), rrd_modify.c(1463)

Define PATH_MAX as _MAX_PATH for MSVC builds

- The max. length of full pathname is 260 under Windows.
  _MAX_PATH is defined in stdlib.h, use:
  #define PATH_MAX _MAX_PATH

Define HAVE_ASPRINTF and HAVE_VASPRINTF (MSVC)

- asprintf.c and vasprintf-msvc.c are compiled in MSVC builds, so
  define HAVE_ASPRINTF and HAVE_VASPRINTF in win32/rrd_config.h

Use Unix line endings for win32-glob

- Restore original line endings for win32-glob.c and win32-glob.h
- Furthermore, RRDtool source files use Unix line endings in general

Remove win32/RCa02816, which is a temporary file

Fix _errno inconsistent dll linkage (Windows)

- Exclude in case of _WIN32 in rrd_snprintf.c:
  extern int errno;
- Fixes warnings:
  MSVC:
  rrd_snprintf.c(583): warning C4273:
  '_errno': inconsistent dll linkage
  errno.h(19): note: see previous definition of '_errno'
  MinGW-w64 (gcc):
  rrd_snprintf.c:583:12: warning: '_errno' redeclared without
  dllimport attribute: previous dllimport ignored [-Wattributes]

Fix rc compiler redefinition warnings (nmake)

- Surround code in win32/rrd_config.h that causes warnings by rc with
  #ifndef RC_INVOKED and #endif. Remark: _MSC_VER is unknown to rc
- Fixes the following warnings for .rc files, e.g.:
  rc /nologo /I./src /fo./win32/rrdtool.res ./win32\rrdtool.rc
  ./win32\rrd_config.h(130) : warning RC4005: 'isinf' : redefinition
  ./win32\rrd_config.h(131) : warning RC4005: 'isnan' : redefinition
- This concerns: librrd-4.rc, rrdtool.rc, rrdupdate.rc, rrdcgi.rc,
  when using the resource compiler (rc) via nmake under Windows

Initialize local variable junk

- Fixes the following compiler warnings (MSVC):
  rrd_modify.c(439) : warning C4700: uninitialized local variable
  'junk' used
  rrd_modify.c(1027) : warning C4700: uninitialized local variable
  'junk' used

Include winsdkver.h for _WIN32_MAXVER

- _WIN32_MAXVER is defined in winsdkver.h
  The include is required in case of MinGW-w64 builds
- Fixes rrd_open.c:11:5: warning:
  "_WIN32_MAXVER" is not defined, evaluates to 0 [-Wundef]
  #if _WIN32_MAXVER >= 0x0602 /* _WIN32_WINNT_WIN8 */

Use _WIN32 instead of WIN32 preprocessor macro

- _WIN32 is the recommended preprocessor macro
- Substitute occurrences of WIN32 with _WIN32 using:
git grep -lw -e '#ifdef WIN32' -e '#ifndef WIN32' -e 'defined(WIN32)' \
-- '*.c' '*.h' '*.xs' | xargs \
sed -i -e 's/#ifdef WIN32/#ifdef _WIN32/g' \
-e 's/#ifndef WIN32/#ifndef _WIN32/g' \
-e 's/defined(WIN32)/defined(_WIN32)/g'

Fix: Cannot rename temporary file to final file

- Fixes the following failing tests under Windows:
  modify1, modify2, modify3, modify4, modify5
- Use CreateFileA() with FILE_SHARE_DELETE in rrd_open.c,
  so that the outfilename can later be replaced by tmpfilename using
  write_rrd() in rrd_create.c