mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alex Clemmer <>
Subject Review Request 53552: Remove unnecessary use of `typename` causing Windows build break.
Date Mon, 07 Nov 2016 19:11:12 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for mesos, Daniel Pravat and Joseph Wu.

Repository: mesos


The use of `typename` in C++ is used to disambiguate between static
members and dependent names that use the same symbol. For example, if
you have some `T::iterator` and `T` happens to have a static member
called iterator, it is necessary to add `typename T::iterator` to
indicate that you want an iterator of `T` rather than to refer to teh
static member `T::iterator`.

In some cases we employ `typename` to make our intention clearer, even
though it is not strictly speaking necessary. While normally a good
habit, in the specific cases we change in this review, it causes MSVC to

This commit will remove these uses.


  src/master/allocator/mesos/hierarchical.cpp c8f9492ee1b69e125a1e841116d22a578a9b524e 




Alex Clemmer

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message