Issue 50 in google-ctemplate: template_modifiers.h unused parameters

[codesite...@google.com]

Status: New
Owner: ----

New issue 50 by pepone.onrez: template_modifiers.h unused parameters
http://code.google.com/p/google-ctemplate/issues/detail?id=50


I get this warning from the unused paramenters in MightModify

from /opt/ctemplate/include/ctemplate/template.h:48,
from Hello.cpp:8:
/opt/ctemplate/include/ctemplate/template_modifiers.h:116: warning: unused
parameter ‘per_expand_data’
/opt/ctemplate/include/ctemplate/template_modifiers.h:116: warning: unused
parameter ‘arg’

virtual bool MightModify(const PerExpandData* per_expand_data,
const std::string& arg) const {
return true;
}

It will be better to get rid of the name of the unused parameters to avoid
this warnning.

virtual bool MightModify(const PerExpandData*,
const std::string&) const {
return true;
}


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

[codesite...@google.com]

Updates:
Status: Started
Labels: Type-Defect Priority-Medium

Comment #1 on issue 50 by csilvers: template_modifiers.h unused parameters
http://code.google.com/p/google-ctemplate/issues/detail?id=50

Sure thing. We'll do this for the next release.

[codesite...@google.com]

Updates:
Status: Fixed

Comment #2 on issue 50 by csilvers: template_modifiers.h unused parameters
http://code.google.com/p/google-ctemplate/issues/detail?id=50

This should be fixed in ctemplate 0.97, just released.

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--

You received this message because you are subscribed to the Google Groups "google-ctemplate" group.
To post to this group, send email to google-c...@googlegroups.com.
To unsubscribe from this group, send email to google-ctempla...@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/google-ctemplate?hl=en.