have InsertLine take a size_t (and pragma 4018 removal)

[GoogleCodeExporter]

Noticed the following comment in template.cc:
// TODO(csilvers): have InsertLine take a size_t

I took a stab at it and figured it would be worthwhile to trace through the
warnings suppressed by pragma 4018 as well.

Original issue reported on code.google.com by josepharmbruster@gmail.com on 15 Nov 2007 at 3:00

Attachments:

• pragma4018.patch

[GoogleCodeExporter]

Note:

line 210 of template_regtest.cc will need repair

Original comment by josepharmbruster@gmail.com on 15 Nov 2007 at 3:07

[GoogleCodeExporter]

Cool, thanks for taking the time to do this.  I'll use this patch as the basis 
for
some more type fixups, which we should expect to see in the next release.

Original comment by csilv...@gmail.com on 15 Nov 2007 at 8:06

• Changed state: Started

[GoogleCodeExporter]

This should be fixed in ctemplate 0.9, which was just released.

Original comment by csilv...@gmail.com on 25 Jan 2008 at 8:32

• Changed state: Fixed