tag:robotlegs.tenderapp.com,2009-10-18:/discussions/problems/406-continued-injection-warnings-with-signalcommandmapRobotlegs: Discussion 2018-10-18T16:35:33Ztag:robotlegs.tenderapp.com,2009-10-18:Comment/107621592011-10-20T20:09:32Z2011-10-20T20:09:32ZContinued Injection warnings with SignalCommandMap<div><p>Hi Bill - which version of SCM are you using? The latest one
should be cleaning up its own injections - this warning was seen in
a prior version but shouldn't be present in the latest.</p>
<p>They actually showed up a hole in the SCM but we did fix it
several months ago. (Mappings were being accidentally left lying
around)</p>
<p>So - I'm thinking you're using a very old version? If not then
let me know because it means the git commits have somehow got
scrambled and the patch has been lost.</p>
<p>I wouldn't go with an old one - the interfaces for Signals have
been in flux and you'll run into compatibility issues.</p>
<p>Stray</p></div>Straytag:robotlegs.tenderapp.com,2009-10-18:Comment/107621592011-10-20T20:30:27Z2011-10-20T20:30:27ZContinued Injection warnings with SignalCommandMap<div><p>Grrrr.</p>
<p>I just made a separate clone of <a href=
"https://github.com/joelhooks/signals-extensions-CommandSignal.git">
https://github.com/joelhooks/signals-extensions-CommandSignal.git</a>
and found differences despite doing a git pull on the original. I
will try again. The SignalExecutorMap works, but has potential
issues with interfaces and garbage collection. It would help if
Joel would put a version readme in the repo.</p></div>billtag:robotlegs.tenderapp.com,2009-10-18:Comment/107621592011-10-20T21:04:02Z2011-10-20T21:04:02ZContinued Injection warnings with SignalCommandMap<div><p>Yep, that did it. No more injection errors.</p>
<p>Thx Stray.</p></div>bill