search

timur-losev / nvidia-texture-tools

Automatically exported from code.google.com/p/nvidia-texture-tools
Other
0 stars 0 forks source link

Array::fill() missing semicolon; #179

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
I'm working from r1354 in Mac OS X 10.8.1 with Xcode 4.4.1/4.5 DP4 using Apple 
LLVM compiler 4.1.

Am I correct in thinking that the following needs a semi-colon terminator or 
not?

--------------------------------------------

    // Fill array with the given value.
    template <typename T>
    void Array<T>::fill(const T & elem)
    {
        fill(m_buffer, m_size, elem)    // notice lack of ;
    }

--------------------------------------------

At least that's what clang's Fixit is suggesting I add.

I've included a patch in case it should be changed.

Original issue reported on code.google.com by markdoum...@gmail.com on 2 Sep 2012 at 11:00

Attachments:

GoogleCodeExporter commented 9 years ago 
This issue was closed by revision r1355.

Original comment by cast...@gmail.com on 5 Sep 2012 at 4:06

GoogleCodeExporter commented 9 years ago 
Thanks for the bug report!

Original comment by cast...@gmail.com on 5 Sep 2012 at 4:07