FGCOM/jansson
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

C++: Rename jansson-impl.hpp to jansson.ipp

Browse Source 
The .ipp suffix is for inlined template implementation code.

While at it, use #ifdef and #ifndef instead of #if defined().
This commit is contained in:
Petri Lehtinen 2010-02-02 21:10:57 +02:00
parent b07e69c37a
commit d1a0c3ffc2
3 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Makefile.am
View File

@ -1,4 +1,4 @@
include_HEADERS = jansson.h jansson.hpp jansson-impl.hpp
include_HEADERS = jansson.h jansson.hpp jansson.ipp
lib_LTLIBRARIES = libjansson.la
libjansson_la_SOURCES = \

8
src/jansson.hpp
View File

@ -3,8 +3,8 @@
// Jansson is free software; you can redistribute it and/or modify
// it under the terms of the MIT license. See LICENSE for details.
#if !defined(JANSSON_HPP)
#define JANSSON_HPP 1
#ifndef JANSSON_HPP
#define JANSSON_HPP
#include <string>
#include <ostream>
@ -296,8 +296,8 @@ inline std::ostream& operator<<(std::ostream& os, const json::Value& value);
inline std::istream& operator>>(std::istream& is, json::Value& value);
// include implementation code
#define IN_JANSSON_HPP 1
#include "jansson-impl.hpp"
#define IN_JANSSON_HPP
#include "jansson.ipp"
#undef IN_JANSSON_HPP
#endif // defined(JANSSON_HPP)

4
src/jansson-impl.hpp → src/jansson.ipp
View File

@ -3,8 +3,8 @@
// Jansson is free software; you can redistribute it and/or modify
// it under the terms of the MIT license. See LICENSE for details.
#if !defined(IN_JANSSON_HPP)
#error "jansson-impl.hpp may only by included from jansson.hpp"
#ifndef IN_JANSSON_HPP
#error "jansson.ipp may only be included from jansson.hpp"
#endif
namespace json {