mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alex Clemmer <>
Subject Re: Review Request 53552: Remove unnecessary use of `typename` causing Windows build break.
Date Sat, 03 Dec 2016 00:06:29 GMT

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

(Updated Dec. 3, 2016, 12:06 a.m.)

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.

Diffs (updated)

  src/master/allocator/mesos/hierarchical.cpp 3b759494071c4cae4b8b7dbcb0028df4146fc30e 




Alex Clemmer

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