[webkit-reviews] review denied: [Bug 25972] Help Disambiguate min/max calls in TimeRanges : [Attachment 30755] Correct build error.

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has denied Brent Fulgham <bfulgham at webkit.org>'s
request for review:
Bug 25972: Help Disambiguate min/max calls in TimeRanges
https://bugs.webkit.org/show_bug.cgi?id=25972

Attachment 30755: Correct build error.
https://bugs.webkit.org/attachment.cgi?id=30755&action=review

------- Additional Comments from Darin Adler <darin at apple.com>
This is close to correct, but not quite.

Since this is a header, it's not appropriate to say "using namespace std" here.
We need to remove that. Adding the include of <algorithm> is also good, but it
needs to be sorted in with the rest of the includes, not a separate paragraph.
And we do indeed need to add the "std::" prefixes.

Further, SMILTime.h should not define min and max functions, since the std
versions will work fine on SMILTime objects.

review- because I'd prefer to see a patch that removes the incorrect "using
namespace std".