Removed ADIOS1 engine's fixed -1 skew to timesteps as now both versions handle them the same way. This fixes the heat transfer example with the adios1 engine.

Issue #363 BPFileReader BeginStep return

BeginStep shouldn't return EndOfStream status after reading last step.

Heat transfer example working as expected.
Tests modified.

Enable and Test writing Attributes in Fortran bindings

Each string in a string_array is limited to 256 in C and Fortran
bindings.

Fix warnings from python and GNU headers.

Due to some squireliness of how python headers work, we need to include
any python binding headers first, before any C library headers.  This is
contrary to the projects coding and style guidelines but necessary
since python.h will otherwise re-define core system feature macros.

* fix-vcs2017-build:
  Silence MSVC TR1 noise
  Fix index OOB error
  Re-enable vs2017

fix: only delete array if it was allocated

Cleanup cmake

 heat transfer example worked with DataManWriter with callback based DataManReader

Fortran bindings

turn off VS 2017 build in appveyor

Fortran bindings support for global variables (with test)
Added Fortran function name in error message to improve debug info
Added parameter (FlushStepsCount) control flushing of BP metadata files
at Write

Working on Fortran WriteRead test