search

danielpalme / IocPerformance

Performance comparison of .NET IoC containers
https://danielpalme.github.io/IocPerformance
Apache License 2.0
881 stars 157 forks source link

Adding IEnumerable to HaveBox Adapter #25

Closed chhr closed 11 years ago

danielpalme commented 11 years ago

Sorry, your code throws a runtime error on my machine. Can you confirm this? "Key not found exception" in method RegisterMultiple

chhr commented 11 years ago

Hi Daniel,

Thank for you mail.

I have the same exception.

Strange, because I have used your benchmark code as test while developing.

I'll be back when I know some more.

Regards, Christian Henrik Reich

On Sun, Oct 6, 2013 at 4:43 PM, Daniel Palme notifications@github.comwrote:

Sorry, your code throws a runtime error on my machine. Can you confirm this? "Key not found exception" in method RegisterMultiple

— Reply to this email directly or view it on GitHubhttps://github.com/danielpalme/IocPerformance/pull/25#issuecomment-25769213 .

chhr commented 11 years ago

Hi Daniel,

I have located the error, and it is not HaveBox

And I can tell ifinjector suffers for there same problem.

It seems that there is a thing with visual studio and nuget.

When I clone your repository and builds it, HaveBox and ifinjector doesn't gets referenced properly. I can see that they are in the package directory, but there is warning sign at the reference.

When via Nuget uninstalls HaveBox and there after reinstall, and then uncomment the line you commented in HaveBoxContainerAdapter, it works.

I also have to reinstall ifinjector, to be able to build the project.

Can you confirm that you refer HaveBox 1.5.0 proberly?

Regards, Christian Henrik Reich

On Sun, Oct 6, 2013 at 4:55 PM, Christian Henrik Reich havehave@gmail.comwrote:

Hi Daniel,

Thank for you mail.

I have the same exception.

Strange, because I have used your benchmark code as test while developing.

I'll be back when I know some more.

Regards, Christian Henrik Reich

On Sun, Oct 6, 2013 at 4:43 PM, Daniel Palme notifications@github.comwrote:

Sorry, your code throws a runtime error on my machine. Can you confirm this? "Key not found exception" in method RegisterMultiple

— Reply to this email directly or view it on GitHubhttps://github.com/danielpalme/IocPerformance/pull/25#issuecomment-25769213 .

chhr commented 11 years ago

Hi (again) Daniel,

This is taken from your current IoCPerformance.csproj

<Reference Include="HaveBox, Version=1.4.0.0, Culture=neutral,

processorArchitecture=MSIL">

False 
  <HintPath>..\packages\HaveBox.1.4.0\lib\net45\HaveBox.dll</HintPath>
</Reference>

It says version=1.4.0.0 where it should have said version=1.5.0.0

Regards, Christian Henrik Reich

On Sun, Oct 6, 2013 at 5:35 PM, Christian Henrik Reich havehave@gmail.comwrote:

Hi Daniel,

I have located the error, and it is not HaveBox

And I can tell ifinjector suffers for there same problem.

It seems that there is a thing with visual studio and nuget.

When I clone your repository and builds it, HaveBox and ifinjector doesn't gets referenced properly. I can see that they are in the package directory, but there is warning sign at the reference.

When via Nuget uninstalls HaveBox and there after reinstall, and then uncomment the line you commented in HaveBoxContainerAdapter, it works.

I also have to reinstall ifinjector, to be able to build the project.

Can you confirm that you refer HaveBox 1.5.0 proberly?

Regards, Christian Henrik Reich

On Sun, Oct 6, 2013 at 4:55 PM, Christian Henrik Reich <havehave@gmail.com

wrote:

Hi Daniel,

Thank for you mail.

I have the same exception.

Strange, because I have used your benchmark code as test while developing.

I'll be back when I know some more.

Regards, Christian Henrik Reich

On Sun, Oct 6, 2013 at 4:43 PM, Daniel Palme notifications@github.comwrote:

Sorry, your code throws a runtime error on my machine. Can you confirm this? "Key not found exception" in method RegisterMultiple

— Reply to this email directly or view it on GitHubhttps://github.com/danielpalme/IocPerformance/pull/25#issuecomment-25769213 .

danielpalme commented 11 years ago

Thanks for your investigations. Now everything should work.