On Fri, Feb 9, 2018 at 3:52 PM, Adam C. Emerson <aemerson@xxxxxxxxxx> wrote:
> On 09/02/2018, Patrick Donnelly wrote:
>> Perhaps it is deduced because inode_t is now templated and has a
>> default template argument?
>
> Yes! Pre C++17 we would have had to use inode_t<> everywhere. Now G++
> and Clang are both doing class template argument deduction, but Clang
> is doing it /slightly wrong/.
Ah, I missed this change. Thanks guys!
>
>> Also, wouldn't "inode_t bar()" be more appropriate than "inode_t
>> bar{}"? Or does the former not work?
>
> inode_t bar() runs into The Most Vexing Parse:
>
> https://en.wikipedia.org/wiki/Most_vexing_parse
>
> --
> Senior Software Engineer Red Hat Storage, Ann Arbor, MI, US
> IRC: Aemerson@OFTC, Actinic@Freenode
> 0x80F7544B90EDBFB9 E707 86BA 0C1B 62CC 152C 7C12 80F7 544B 90ED BFB9
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
