yes I agree, that would be nice for non-generate/import guys to not get the .rhtml expansion. However I would like to hold on that till after the release if that is ok since it is internal change.<br><br><div><span class="gmail_quote">
On 6/19/06, <b class="gmail_sendername">Deb Lewis</b> <<a href="mailto:djlewis@acm.org">djlewis@acm.org</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div>
<div></div><div><span class="q">
<div dir="ltr" align="left"><font color="#0000ff" face="Arial" size="2"><span>>> <font color="#000000" face="Times New Roman" size="3">Your keyword expander changes look good, lets go ahead and turn those
on.</font></span></font></div>
<div dir="ltr" align="left"><font color="#000000" face="Times New Roman" size="3"><span></span></font> </div></span></div><div>
<div dir="ltr" align="left"><font color="#0000ff" face="Arial" size="2"><span>ok, I'll flip the toggle switch over the "new and
improved version" and clean out the transition kruft when I have time
tonight.</span></font></div>
<div dir="ltr" align="left"><font color="#0000ff" face="Arial" size="2"><span></span></font> </div>
<div dir="ltr" align="left"><font color="#0000ff" face="Arial" size="2"><span>note to selves: directives.html need a section added to
describe this new keyword expansion feature.</span></font></div>
<div dir="ltr" align="left"><font color="#0000ff" face="Arial" size="2"><span></span></font> </div>
<div dir="ltr" align="left"><font color="#0000ff" face="Arial" size="2"><span>one other possible tweak: maybe either a separate
flavor of expand_keywords or a defaulted parm so that the extra checking for
whether to automatically append .rhtml extension is only incurred by directives
that need that functionality? (i.e., generate/import
guys)</span></font></div>
<div dir="ltr" align="left"><font color="#0000ff" face="Arial" size="2"><span></span></font> </div>
<div dir="ltr" align="left"><font color="#0000ff" face="Arial" size="2"><span>this would need to be hooked up in the proper places in
parser/analyzer, so I hadn't pursued that on first round of revision
proposal.</span></font></div>
<div dir="ltr" align="left"><font color="#0000ff" face="Arial" size="2"><span></span></font> </div>
<div dir="ltr" align="left"><font color="#0000ff" face="Arial" size="2"><span>~ Deb</span></font></div></div>
</div><br>_______________________________________________<br>Masterview-devel mailing list<br><a onclick="return top.js.OpenExtLink(window,event,this)" href="mailto:Masterview-devel@rubyforge.org">Masterview-devel@rubyforge.org
</a><br><a onclick="return top.js.OpenExtLink(window,event,this)" href="http://rubyforge.org/mailman/listinfo/masterview-devel" target="_blank">http://rubyforge.org/mailman/listinfo/masterview-devel</a><br><br></blockquote>
</div><br>