GEODE-10218: Allow server debug (#971)

* Remove unused List<Locator>
* Show withDebugAgent() usage in cli

GEODE-10218: Allow server debug (#954)

* Support DebugAgent in clicache tests
* Support DebugAgent in cppcache tests
* Fix StartLocators bug
* Use builder pattern in Cluster, Locator and Server classes

GEMNC-509: Rename .NET client to .NET Framework (#965)

Co-authored-by: Max Hufnagel <mhufnagel@vmware.com>

GEODE-10259: Update protocol ordinal to 1.14.0 (125) (#962)

* Fixes ClientProxyMembershipID for 1.1.0+
* Fixes TcrConnection for 1.5.0+
* Fixes TXCommitMessage for 1.7.0+
* Updated ClientHealthStats for 1.9.0+
* Updated QueueConnectionRequestTest for 1.14.0+

GEODE-10227: Remove redundant sendRequestForChunkedResponse (#957)

* Remove redundant sendRequestForChunkedResponse
* Remove vestigial comment

GEODE-10074: Remove ACE remainings (#928)

- Removed all the ACE includes along the repository
 - Removed ACE from CMake project.
 - Removed ACE dependency.
 - There were a couple of files in which ACE's strcmp was used.
   So, its usage was replaced to to std::string::operator==
 - Removed non-necessary ACE's getcwd calls.

GEODE-9921: Rename .NET to .NET Framework (#960)

GEODE-10241: disable re-auth test until we can implement properly (#961)

GEODE-10131: Remove set but unused variables (#947)

* Remove set but unused variables

- On AppleClang 13.1.6.13160021, the warning -Wunused-but-set-variable
  is appears as an error.  This allows compilation on AppleClang on
  MacOS 12.3

Authored-by: M. Oleske <michael@oleske.engineer>

Fix lgtm ubuntu break (#958)

- related to the earlier ubuntu break, so applying the same fix

Authored-by: M. Oleske <michael@oleske.engineer>

Bump puma from 5.6.2 to 5.6.4 in /docs/geode-native-book-dotnet (#956)

Bumps [puma](https://github.com/puma/puma) from 5.6.2 to 5.6.4.
- [Release notes](https://github.com/puma/puma/releases)
- [Changelog](https://github.com/puma/puma/blob/master/History.md)
- [Commits](https://github.com/puma/puma/compare/v5.6.2...v5.6.4)

---
updated-dependencies:
- dependency-name: puma
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump puma from 5.6.2 to 5.6.4 in /docs/geode-native-book-cpp (#955)

Bumps [puma](https://github.com/puma/puma) from 5.6.2 to 5.6.4.
- [Release notes](https://github.com/puma/puma/releases)
- [Changelog](https://github.com/puma/puma/blob/master/History.md)
- [Commits](https://github.com/puma/puma/compare/v5.6.2...v5.6.4)

---
updated-dependencies:
- dependency-name: puma
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

GEODE-10120: Bump ruby dependencies (#945)

* Bump addressable from 2.7.0 to 2.8.0 in /docs/geode-native-book-cpp

Bumps [addressable](https://github.com/sporkmonger/addressable) from 2.7.0 to 2.8.0.
- [Release notes](https://github.com/sporkmonger/addressable/releases)
- [Changelog](https://github.com/sporkmonger/addressable/blob/main/CHANGELOG.md)
- [Commits](https://github.com/sporkmonger/addressable/compare/addressable-2.7.0...addressable-2.8.0)

---
updated-dependencies:
- dependency-name: addressable
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
* Bump addressable from 2.7.0 to 2.8.0 in /docs/geode-native-book-dotnet

Bumps [addressable](https://github.com/sporkmonger/addressable) from 2.7.0 to 2.8.0.
- [Release notes](https://github.com/sporkmonger/addressable/releases)
- [Changelog](https://github.com/sporkmonger/addressable/blob/main/CHANGELOG.md)
- [Commits](https://github.com/sporkmonger/addressable/compare/addressable-2.7.0...addressable-2.8.0)

---
updated-dependencies:
- dependency-name: addressable
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Remove spurious comma (#951)

Packer builds for Windows were failing because JSON parsing failed.

Ubuntu build break fix, 2nd attempt (#953)

* Fix Ubuntu build break due to cmake upgrade
- New cmake version changes treatment of CMAKE_INSTALL_LIBDIR variable,
  which now evaluates to different things for xerces build and main
  client build.  As a result, we build Xerces, then go looking to link
  to it in the wrong place.  Hard-coding the path to just 'lib' for now
  appears to fix the Ubuntu issue without breaking anything else.

* 2nd attempt to fix Ubuntu build break
- First attempt broke RHEL flavors, and only fixed the first dependent
  library on Ubuntu - gtest was still looked for in the wrong place
- It appears cmake has special code on Ubuntu to use the wrong lib dir
  iff CMAKE_INSTALL_PREFIX is set to /usr or /usr/local, so we'll set it
  to something else and see if that does or doesn't break anything else.

Fix Ubuntu build break due to cmake upgrade (#952)

- New cmake version changes treatment of CMAKE_INSTALL_LIBDIR variable,
  which now evaluates to different things for xerces build and main
  client build.  As a result, we build Xerces, then go looking to link
  to it in the wrong place.   It appears cmake has special code on Ubuntu to use the wrong lib dir
  iff CMAKE_INSTALL_PREFIX is set to /usr or /usr/local, so setting it to something else seems the safest approach.

GEODE-10058: Remove netcore and cbindings (#950)

* Remove c-bindings
* Remove netcore
* Remove dotnetcore sdk
* Remove c-bindings and netcore from CMakeLists.txt
* Remove netcore tests from CI

Co-authored-by: Blake Bender <bblake@vmware.com>

GEODE-10101: Bump Geode version to 1.14.4 (#948)

Native client hardcodes Geode version to test with in several places.
Update those variables to latest-and-greatest apache-geode 1.14.4

Fix regex used to read server handshake requests (#946)

Fix regexes in server messages (#944)

Gnmsg parse server handshake response (#939)

* Add timestamp, tid, direction, and Type to server handshake request

* Parsing server handshake response in gnmsg
- Added logging to TcrConnection to dump handshake response bytes
- Cleaned up some code in TcrConnection::initTcrConnection, to help
  clarify what's going on
- Renamed TcrConnection member variables to conform to standard

Add shorts and longs to read_cacheable (#940)

GEODE-10085: Don't start JmxManager if already running (#936)

* Don't start JmxManager if already started
* Fix broken StartLocators() and StartServers()

GEODE-10082: Fixes mixing of DSCode and DSFid enums. (#933)

Moves InterestResultPolicy and ClientProxyMembershipId to DSFid.

Replaces eroneous CacheableEnum with PDX_ENUM.

Allow --handshake and --messages flags to coexist in gnmsg (#935)

GEODE-10062: Update Native Client Docs to minimize redirects - additions and corrections (#934)

gnmsg now parses all observed locator requests and responses (#932)

- Also remove duplicate values from ds_codes enum
- geode-native has these wrong right now in the C++ code, so they came
  across to Python this way.

GEODE-10062: Update Native Client Docs to minimize redirects (#931)

GEODE-10073: Fix regex issue on Linux systems (#929)

- Hex dump of a pointer on Linux is prefixed with '0x', so need to
  allow x or X character

GEODE-10073: Print bytes of ClientConnectionRequest/Response in logs (#927)

* Print bytes of ClientConnectionRequest/Response in logs
* Dump the bytes at debug log level, so we can parse
* Also added parsing of these to gnmsg tool

Co-authored-by: Blake Bender <bblake@vmware.com>

GEODE-9325: Replace old ITs worker spawner (#813)

- Replaced references to ACE_Process realated to old ITs process
   spawners.
 - Added Boost program_options project, given that was needed to fulfill
   above changes.
 - Fix Windows execution whenever a test failed. Apparently if not all
 the processes have terminated ctest does not exit its execution and due
 to the fact that there is a worker in charge of managing the cluster
 processes, if the coordinator terminates all of the workers, it's not
 guaranteed that the cluster is stopped.
 - Reversed order in which cleanups are called in order to ensure proper
 teardown.

GEODE-10065: Add missing colon in regexp (#926)

GEODE-9327: Remove ACE networking code references (#815)

 - Replaced method implementation to get a random port in old ITs framework.
 - Replaced convertHostToCanonicalForm implementation.
 - Changed ClientProxyMembershipID address serialization.
 - Removed unused PoolFactory::addCheck method.
 - Modified QueueConnectionRequestTest test to pass after the changes.
 - Fixed min/max macros windows compilcation
 - Changed ClientProxyMembershipID member attribute names to match style
   guidelines.
 - Changed HostStatSampler member attribute names to match style
 guidelines.
 - Changed Utils::convertHostToCanonicalForm to make the method more
   readable.

GEODE-9326: Replace ACE_Get_Opt (#814)

 - Replaced ACE_GET_Opt by Boost alternative.
 - Due to the include changes there was a compilation error in fw_spawn,
   which is solved by this commit.
 - Fixed FD_SETSIZE to 1024 for Win32 to avoid coredump in ACE_Process

GEODE-9324: Remove ACE_Task references (#812)

 - Removed all references to ACE_Task
 - Changed variable names to comply with styleguides

GEODE-9268: Fix PdxInstance handling after cluster restart (#806)

* GEODE-9268: Fix PdxInstance handling after restart

 - In scenarios where an PdxInstance is obtained and later used, it
   might happen a coredump if the PdxTypeRegistry is cleaned up after the
   PdxInstance is obtained. This happens on those scenarios where
   redundancy is completly lost.
 - This change refactors PdxInstance handling so in all cases, the PdxType
   used is the one owned by the PdxInstance.
   Whenever the PdxInstance is to be written, a check is executed to
   ensure the cluster is aware of the PdxType and if not, register it.
 - Removed PdxInstance serialization retries, as with the new approach
   is not needed anymore.
 - Removed UnknownPdxTypeException exception. Instead whenever a PdxType
   is requested and not present, an IllegalStateException exception is
   thrown, as it happens in the Java client.
 - Now PdxInstance is not serialized whenever created. Instead, it's PDX
   byte stream is generated on-demand. Note that the PdxInstance will be
   serialized before being put into a server, as it was done before this
   change.
 - Fixed TcrMessage deserialization whenever a PdxType is requested by
   its ID and no PdxType was found.
 - Fixed incPdxInstanceCreations so it's incremented strictly whenver a
   PdxInstance is created, and not whenever a PdxInstance is
   deserialized.
 - Fixed IT PdxTypeRegistry cleanupOnClusterRestart logic and renamed to
   cleanupOnClusterRestartAndPut. This test was supposed to verify that
   if a PdxInstance is created, and after that the cluster is restarted,
   there is no coredump while writting it to a region. Instead it was
   creating a PdxInstance before and after the cluster, but it has been
   fixed to work as initially intended.
 - Created a new IT PdxTypeRegistry cleanupOnClusterRestartAndFetchFields
   to verify the issue described in the first bullet is not causing a coredump.
 - Removed old IT testThinClientPdxInstance TS as there is a equivalent new
   IT TS named PdxInstanceTest.
 - Fixed some ITs to work accordingly the the new code.
 - Fixed PdxInstanceImplTest.updatePdxStream to work accordingly with
   the code changes.

* GEODE-9268: Revision 1

 - Refactored member attributes for NestedPdxObject classes in order to
   follow style guidelines.
 - Refactored member attributes for PdxInstanceImpl class in order to
 follow style guidelines.
 - Moved default bytes variables from static to constants inside an anon
   namespace within PdxInstanceImpl cpp file.
 - Renamed 'pft' variables to 'field' inside PdxInstanceImpl in order to
   make the code more readable.
 - Changed return of PdxLocalWriter::getPdxStream to use the explicit
   constructor of std::vector<uint8_t>
 - Solved PdxInstanceTest.testNestedPdxInstance asserts description
   mismatch.
 - Used binary_semaphore inside PdxTypeRegistryTest insteda of an
   in-place CacheListener in order to listener for cluster start/stop
   events.
 - Revert back changes to the OQL used inside
   PdxTypeRegistryTest.cleanupOnClusterRestartAndPut
 - Added test descriptions for both tests inside PdxTypeRegistryTest.

* GEODE-9268: Revision 2

 - Addressed several others assert descriptions mismatch.

GEODE-10040: Increase wait timeout for gfsh (#924)

GEODE-10004: support older geode versions (#917)

* Check for class existence instead of version.

GEODE-100043: Solve TransactionCleaningTest errors (#923)

- From time to time this test was failing with the error "Concurrent
   modification in the cache". Thing is this test was not intended to
   check concurrent modifications.
 - In GEODE-9322 this test was modified to avoid a race-condition
   whenever restarting the server. To do so, it was needed to enable
   notification subscription and change the region shortcut to
   CACHE_PROXY. In doing so, concurrent checks are also enabled, and due
   to some strange race-condition this is occasionally leading to the exception.
 - Concurrency checks were disabled for the region to avoid the
   race-condition to happen at all

Add license text to some doc sources (#922)

GEODE-9959: Add FQDN in log upon SSL missconfig (#904)

- Whenever there is an SSL missoconfiguration while trying to reach a
   locator, an AuthenticationRequiredException is thrown and a log is
   written. Currently FQDN of the locator is not logged.
 - This commit adds the locator FQDN in order to further troubleshoot
   this kind of scenarios.
 - Also, in order to ease implementation, toString method was
   implemented for ServerLocation.

GEODE-10030: Remove obsolete cross-reference in geode-native user docs (#921)

GEODE-9960: Remove strong name signing support for .net assemblies (#905)

* Remove strong name signing support for .net assemblies
- This has caused problems with build pipelines, and is no longer used by .net runtimes
- Make internals visible to test DLL

GEODE-10016: Add map of threadId to threadName (#918)

* Save threadName to thread_local storage
* Per review, get/setThreadName doesn't need tid
* Add threadName in parens after tid
* Update gnmsg for threadName
* Move accessors to Log class
* Inline threadNames instead of constants
* Remove getThreadName()

GEODE-9323: Remove ACE from tests/cpp projects (#811)

* GEODE-9323: Remove ACE from tests/cpp projects

 - Removed all ACE references from tests/cpp projects.
 - Also moved SerializationRegistry inline functions to the
   source file in order to solve compilation.

* GEODE-9323: Empty commit to trigger CI

GEODE-9896: bump sqlite-fx to 1.0.115.5 (#916)

GEODE-9896: Bump sqlite to 3.37.2 (#914)

Authored-by: M. Oleske <michael@oleske.engineer>

GEODE-9896: Bump ace to 7.0.6 (#913)

- fix building on macos monterry

Authored-by: M. Oleske <michael@oleske.engineer>

Bumping Geode version to 1.14.3 for CI (#912)

Fix typo in gfsh command ('hreshold' vs 'threshold') (#911)

Co-authored-by: Blake Bender <bblake@vmware.com>

Eliminate duplicate REGISTER_INTEREST message (#906)

- first endpoint in m_redundantEndpoints was originally supposed to
be left out of the for loop.  This broke during a refactor roughly three years ago, moving from old-school iterator to a ranged for loop.

Co-authored-by: Blake Bender <bblake@vmware.com>

Bumping Geode version to 1.14.2 for CI (#902)

Bumping Geode version to 1.14.1 for CI (#900)

Fix to disable auto updates on rhel8 (#898)

gnmsg - decode REGISTER_INTEREST message (#895)

* Make log-rolling work with relative paths
* Don't report individual chunk stats, just stats for the complete
response.
* Parse REGISTER_INTEREST client message details
* Tiny performance improvement to regex parsing
  - Really need to spend time tuning these

GEODE-9804: new register interest tests (#894)

* Fixes incorrect fetching of initial entries.
* Makes InterestResultPolicy a proper enum.
* Adds integration benchmark for register interest.
* Fixes for MSVC errors.
* Don't ask for keys if not caching locally.
* Replace sleep with CacheListener events.
* Use latches for syncrhonization with listener.
* Change boost loglevel to warning.
* Encapsulate synchronization in CountdownCacheListener.

Co-authored-by: Jacob Barrett <jbarrett@pivotal.io>

GEODE-9741: Fixes Boost boostrap command. (#890)

GEODE-9741: Pass full toolkit version to external build systems. (#889)

* Require at least CMake 3.19.7 for 3 component toolkit version support.
* LGTM downloads latest CMake.

GEODE-9741: Use v142 Toolchain in Visual Studio 2019 CI Pipeline (#884)

* Add v142 to VS 2019 and build with it
* Use explicit versions
* Use lowest versions for tools & sdk

make sure geode version matches from start to fan-in (#887)

Fix Ubuntu images in CI (#886)

* Don't perform upgrade when creating instance
- This appears to be breaking SSH connectivity

GEODE-9553: remove all remaining printf variations from C++ code (#885)

GEODE-9639: Removed the use of std::unary_function. (#874)

* Removed remaining deprecations that can be removed.

GEODE-9719: Start/Stop cluster in a test fixture (#881)

* Start/Stop cluster in a test fixture
* Create an authRegion in same cluster
* Turn off cluster output after all tests run

GEODE-9553: remove sprintf usage in test utility code (/tests/cpp) (#883)

GEODE-9688: net-core builds only for x64 (#878)

* GEODE-9688: net-core builds only for x64
* Fix path for install
* Specifiy platform for test runs

Fix build break (#882)

- changes to Cluster ctor crossed paths on the way into develop with a
  change that added some new tests

GEODE-9600: Add Cluster Support to NetCore Test Framework (#880)

* Add cluster support to netcore-integration-test and netcore-session integration-test
* Add top level solution for netcore
* Add programmatic cluster files in shared folder for use by both test projects
* Address Rat check issue: Add using statement to new Process
* Remove gfsh based cluster scripts, now supported in cluster.cs, etc.
* Update refs to Cluster.cs, etc. in project files
* cmake configure Config.cs.in so environment vars are not required
* Remove old netcore solution file
* Cleanup per review: Geode .Net Core, not .net Core

GEODE-9634: Workaround for failing test. (#872)

* Updates test.
* Fixes hostname issues with tests.
* Fix default locator address init.

GEODE-9591: Don't retry function when Function.isHA=false (#866)

* GEODE-9591: Don't retry function when Function.isHA=false

Fault:
Client clears result by calling ResultCollector::clearResult and
re-executes the function even thought Function.isHa() is set to false.

Fix:
Client will not clear result and retry the function, but it will fail with the
received exception from server.

Added integration test:
1.FunctionExecutionTest.testThatFunctionExecutionThrowsExceptionNonHAWithFilter
2.FunctionExecutionTest.testThatFunctionExecutionThrowsExceptionNonHA

The integration test reproduces the problem in a following way:

1. Populate region so that all buckets are created
2. Shutdown one of the three available servers
3. Peforme rebalance operation, so that buckets are recovered on running
servers
4. Execute the function. Function execution will fail with the
InternalFunctionInvocationTargetException exception, because client will
try to execute single hop function with old PR meatada information (PR
metadata received before the rebalance operation).

* ASSERT_THROW used instead of try-catch

* Check whether isHA==false before clearing results

It is possible that function exeuction fail on multiple servers, and in
that case only first exception will be thrown, and any subsequent
exeception will be ignored. Additionally result should not be cleared in
any case when isHa==false.

* Rebalance cmd was not executed

Test cases are changed to do rebalance operation
after new server is introduced in cluster.

* Client metadata wasn't complete when function is executed

Since client metadata was not complete the function was not exectued in
a single hop manner, and therfore exeption is not reproduced.

This commit is fixed in a way that now client is connected to only
one server. This way client metadata will be updated more frequently,
and client metadata will be complete when function is executed.

GEODE-9553: remove usage of sprintf from tests (#879)

GEODE-9631: New tests for reauthentication (#876)

* New tests for reauthentication and durability
* Add new SimulatedExpirationSecurityManager for reauthentication testing

GEODE-9608: Remove Release config, add Opimization and pdb (#877)

Add a fan-in job and print out the properties of the Geode build used for testing (#875)

GEODE-9569: Native client UG build script keeps doc after generating it (#873)

GEODE-9553: remove sprintf and snprintf from geode native (#871)

* GEODE-9553: Replace instances of sprintf and snprintf in geode native
- Occasionally one of these will pop up as an error in a new compiler,
  and really they have no place in a modern C++ codebase.
- Just the product code (mostly) for now, we'll do test code in another PR

GEODE-9576: Fix for single-hop function execution (#864)

* GEODE-9576: Fix for single-hop function execution

Fault:
"InternalFunctionInvocationTargetException: Multiple target nodes
found for single hop operation" occurs on server when executing
function in a single hop manner for all buckets during the period when
client bucket metadata doesn't contains all buckets locations.

Fix:
The client will execute function in a non single-hop manner until it
recevies all buckets locations. This solution is aligned with java
client.

* Updates after comments

* Fix for clang-tidy errors

* Fix for clang-tidy

Use old MOCK_METHOD1 because clang-tidy reports error for the new macro.

GEODE-9609: Fix Warning in SessionState (#869)

GEODE-9637: Disable Wan deserialization test (#870)

- Just want a green CI until GEODE-9634 is resolved.

Co-authored-by: Blake Bender <bblake@vmware.com>

GEODE-9549: enable netcore tests in CI (#867)

* GEODE-9549: Enable .net core tests in CI
- Enable test running job in yml
* GEODE-9549: Build NetCore and NetCore.Test projects individually
- This simplifies the cmake config, and makes the post-build file copy
  steps more reliable
* GEODE-9549: Copy C bindings lib to binary dir for tests
- Give Java project a reasonable name
* GEODE-9549: Use ctest to run netcore tests
- Eliminates need for start/stop scripts, all done via ctest now
* GEODE-9549: Make .net core project build in sequence
- Still getting sharing violations on parallel builds, so need to ensure
  none of these build at the same time.

Co-authored-by: Jacob Barrett <jabarrett@vmware.com>

GEODE-9189: Upgrade dependencies.

Boost to 1.76.
ACE to 7.0.3.
GTest 1.11.0.
Benchmark 1.6.0.
SQLite to 3.36.0

Removed unused colored_printing, which depends on internals of gtest.

GEODE-7282: Geode Native Client .NET guide: Fix capitalization issues (#863)

Bumping Geode version to 1.14.0 for CI (#865)

GEODE-9551: Fixes API return and parameter types. (#860)

GEODE-9464: Add Asp.Net Core session state sample application  (#856)

* Add AspNetCore GeodeSession Sample
* Create region for sessionState in startserver
* Add Apache license header

Fix buffer overflow gripes in RHEL-8 build (#861)

- RHEL-8 CI images appear to have picked up a new compiler that's
  flagging some broken sprintf code in a couple of legacy tests.

GEODE-9550: Native Client user guide: warn against floating point values in keys (#857) (#859)

Fixes disable of LTO on RHEL8. (#858)

Support "rolled" log files in gnmsg (#854)

- With new parameter --rolled, gnmsg will expect file of the form
  <<basename>>-<<number>>.<<ext>>, and will scan the same directory for files of
  the same form with incrementing numbers until it runs out.  It will
  then attempt to open <<basename>>.<<ext>>, which should be the last file
  geode-native was logging to at the time of app exit.  Output will be a
  single consecutive JSON array of message structures, handy for
  post-processing.
- single file behavior remains the same as before

Disables LTO on RHEL-8 (#855)

* Adds LTO and sanitizer options.
* Adds checks for some sanitizer options.
* Adds options to enable sanitizers.
* Extracts checks for LTO.
* Adds options to disable LTO.
* Disables IPO on RHEL-8 because GCC 8.4.1 is broken.

GEODE-9524: Make library name and directory name consistent (#853)

Co-authored-by: Matthew Reddington <mreddington@vmware.com>

GEODE-9511: Update ACE cmake config for VS2019 (#847)

GEODE-9356: Disabled netcore-tests until working.

GEODE-9356: Fixes dotnet install.

GEODE-9356: Fixes test path.

Unlock develop, attempt #3 (#851)

Unlock develop branch, 2nd attempt (#849)

* Unlock develop branch, 2nd attempt
- Just make all gates optional instead, and require 0 reviews

* Also make source code checks optional

GEODE-9356: Making netcore-tests run on Windows and Linux (#844)

(Temporarily) Unlock develop (#846)

Fix formatting (#845)

We took an update to the clang tools a couple days ago, and the new clang-format disagrees with the previous version on a couple of files.