[llvm-commits] [llvm] r120998 - /llvm/trunk/include/llvm/Support/FileSystem.h

[Steven Watanabe]

AMDG

On 12/6/2010 2:48 AM, Frits van Bommel wrote:
> Author: fvbommel
> Date: Mon Dec  6 04:48:11 2010
> New Revision: 120998
>
> URL: http://llvm.org/viewvc/llvm-project?rev=120998&view=rev
> Log:
> Fix clang warning: "extra ';' inside a class [-pedantic]".
>
> Modified:
>      llvm/trunk/include/llvm/Support/FileSystem.h
>
> Modified: llvm/trunk/include/llvm/Support/FileSystem.h
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Support/FileSystem.h?rev=120998&r1=120997&r2=120998&view=diff
> ==============================================================================
> --- llvm/trunk/include/llvm/Support/FileSystem.h (original)
> +++ llvm/trunk/include/llvm/Support/FileSystem.h Mon Dec  6 04:48:11 2010
> @@ -607,7 +607,7 @@
>     }
>
>     const directory_entry&operator*() const { return CurrentEntry; }
> -  const directory_entry *operator->() const { return&CurrentEntry; };
> +  const directory_entry *operator->() const { return&CurrentEntry; }
>
>     bool operator!=(const directory_iterator&RHS) const {
>       return CurrentEntry != RHS.CurrentEntry;

Why does this cause a warning with -pedantic?  It looks like
the standard allows it:

member-declaration:
    function-definition ;opt

In Christ,
Steven Watanabe