On Wed, Mar 13, 2013 at 7:36 AM, Otto van der Schaaf <
notifications@github.com> wrote:

> In src/ngx_rewrite_options.cc:
>
> > @@ -16,18 +16,21 @@
>
> probably intended, but I thought I'd mention it, cpplint complains about
> dynamic_cast usage:
>
> Do not use dynamic_cast<>. If you need to cast within a class hierarchy,
> use static_cast<> to upcast. Google doesn't support RTTI.
>

cpplint is wrong here (perhaps the external version is out of date?).
Here is what the style guide says:
"When the logic of a program guarantees that a given instance of a base
class is in fact an instance of a particular derived class, then a
dynamic_cast may be used freely on the object. "