Differential D39166 Diff 120346 include/clang/AST/InlineDeclMembers.h

Changeset View

Standalone View

include/clang/AST/InlineDeclMembers.h

				//===- InlineDeclMembers.h - Decl.h Members that must be inlined -- C++ --==//
				//
				// The LLVM Compiler Infrastructure
				//
				// This file is distributed under the University of Illinois Open Source
				// License. See LICENSE.TXT for details.
				//
				//===----------------------------------------------------------------------===//
				//
				// This file defines inline members of classes defined in Decl.h that have
				// (circular) dependencies on other files and should only be included
				aaron.ballmanUnsubmitted Done Reply Inline Actions This comment seems incorrect. aaron.ballman: This comment seems incorrect.
				// in .cpp files (as opposed to .h).
				//
				//===----------------------------------------------------------------------===//

				#ifndef LLVM_CLANG_AST_INLINEDECLMEMBERS_H
				#define LLVM_CLANG_AST_INLINEDECLMEMBERS_H

				#include "clang/AST/Decl.h"
				#include "clang/AST/DeclCXX.h"
				namespace clang {
				aaron.ballmanUnsubmitted Done Reply Inline Actions Spurious newline. Because this functionality will always be for inline Decl members, I think it would make more sense to put the declarations into `namespace clang` rather than use a qualified name. aaron.ballman: Spurious newline. Because this functionality will always be for inline Decl members, I think…

				inline bool FunctionDecl::willHaveBody() const {
				assert(!isa<CXXDeductionGuideDecl>(this) &&
				"must not be called on a deduction guide since we share this data "
				"member while giving it different semantics");
				return WillHaveBody;
				}

				inline void FunctionDecl::setWillHaveBody(bool V) {
				assert(!isa<CXXDeductionGuideDecl>(this) &&
				"must not be called on a deduction guide since we share this data "
				"member while giving it different semantics");
				WillHaveBody = V;
				}
				aaron.ballmanUnsubmitted Done Reply Inline Actions Another spurious newline. aaron.ballman: Another spurious newline.

				aaron.ballmanUnsubmitted Not Done Reply Inline Actions Whitespace is incorrect here. aaron.ballman: Whitespace is incorrect here.
				faisalvAuthorUnsubmitted Not Done Reply Inline Actions not sure I follow? faisalv: not sure I follow?
				} // namespace clang

				#endif // LLVM_CLANG_AST_INLINEDECLMEMBERS_H