approvals / ApprovalTests.cpp

Native ApprovalTests for C++ on Linux, Mac and Windows
https://approvaltestscpp.readthedocs.io/en/latest/
Apache License 2.0
318 stars 51 forks source link

Move to #pragma once for include guards #60

Closed claremacrae closed 4 years ago

claremacrae commented 4 years ago

So navigation of macros in Source Trail is more useful.

claremacrae commented 4 years ago

Our current released single header begins like this:

// Approval Tests version v.6.0.0
// More information at: https://github.com/approvals/ApprovalTests.cpp
#include <string>
#include <fstream>
...
#include <exception>
#include <map>
 // ******************** From: Blocker.h
#ifndef APPROVALTESTS_CPP_BLOCKER_H
#define APPROVALTESTS_CPP_BLOCKER_H
...
#endif

If we change to #pragma once, we'll end up with multiple of those lines in the single header... that may not matter, but it will look odd...

claremacrae commented 4 years ago

This has been done