[PATCH] D33447: clang-format: add option to merge empty function body

[Francois Ferrand via Phabricator via cfe-commits]

Typz added a comment.

Right, I was not clear enough: this is indeed done only if everything does not fit on one line.

There is an exemple in facebook's coding style:

  MyClass::MyClass(const Class* cls, const Func* func, const Class* ctx)
    : m_cls(cls)
    , m_func(func)
    , m_ctx(ctx)
    , m_isMyConditionMet(false)
  {}

And this happens (with the extra space) in the same kind of situation in Mozilla's code:

  nsSameProcessAsyncMessageBase::nsSameProcessAsyncMessageBase(JS::RootingContext* aRootingCx,
                                                               JS::Handle<JSObject*> aCpows)
    : mRootingCx(aRootingCx)
    , mCpows(aRootingCx, aCpows)
  { }


https://reviews.llvm.org/D33447