Differential D39166 Diff 119791 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 the Decl subclasses.
				aaron.ballmanUnsubmitted Done Reply Inline Actions This comment seems incorrect. aaron.ballman: This comment seems incorrect.
				//
				//===----------------------------------------------------------------------===//

				#ifndef LLVM_CLANG_AST_INLINEDECLMEMBERS_H
				#define LLVM_CLANG_AST_INLINEDECLMEMBERS_H

				#include "clang/AST/Decl.h"
				#include "clang/AST/DeclCXX.h"


				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 clang::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 clang::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.
				#endif //LLVM_CLANG_AST_INLINEDECLMEMBERS_H
				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?