I'd be more than happy to incorporate those changes. I'd most probably
put it on an option and make a new package.
Cheers,
Jim Tilander
--
Beware of architect astronauts.
/j
http://niftyplugins.googlecode.com/svn/trunk/Build/Experimental_NiftyPerforce.msi
and take it for a spin. Thanks for the patch!
/j
--- code ---
// [jt] This handler checks for things like paste operations. In
theory we should be able to remove the handler above, but
// I can't get this one to fire reliably... Wonder how much these
handlers will slow down the IDE?
void OnLineChanged(TextPoint StartPoint, TextPoint EndPoint, int Hint)
{
if ((Hint & (int)vsTextChanged.vsTextChangedNewline) == 0 &&
(Hint & (int)vsTextChanged.vsTextChangedMultiLine) == 0 &&
(Hint & (int)vsTextChanged.vsTextChangedNewline) == 0 &&
(Hint != 0))
return;
if (Singleton<Config>.Instance.autoCheckout &&
m_application.ActiveDocument.ReadOnly)
P4Operations.EditFile(m_outputPane, m_application.ActiveDocument.FullName);
}
--- code ---
Ok. So I think I remember that I had a breakpoint in the
function-entry and it didn't fire at all times I would expect it to. I
think that was the extent to which I worried about it since I only put
this code in here to begin with at a feature request (I'm not eating
my own dogfood here for this particular feature :).
If you've got it working reliably, that's great!
I'm unsure as to the behavior you want from the code change, but a
smaller (like your one-liner) is always better in my eyes :)
Cheers,
Jim