[libcxx-commits] [PATCH] D88727: vector (iterator, iterator) constructor doesn't deduce second arg

[kamlesh kumar via Phabricator via libcxx-commits]

kamleshbhalui added a comment.

In D88727#2308363 <https://reviews.llvm.org/D88727#2308363>, @miscco wrote:

> So the actual problem in the bug report is that the first argument cannot be used to deduce the type.
>
> I think it is not correct to remove the type restrictions here, they are there for a reason.

The only restriction is that type should be input iterator and that is still in force.

> That said is there any reason the constraint is placed where it is and not in the template header where I would have expected it?

That is because the declaration and definition are at different places.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88727/new/

https://reviews.llvm.org/D88727