Re: Issue 776 in google-guice: Avoid unnecessary object creation if .withSource doesn't change the source

2013-10-22 Thread google-guice
Comment #3 on issue 776 by mccu...@gmail.com: Avoid unnecessary object creation if .withSource doesn't change the source http://code.google.com/p/google-guice/issues/detail?id=776 As mentioned above, when replaying/applying bindings using the SPI (for example when using Modules.override

Re: Issue 776 in google-guice: Avoid unnecessary object creation if .withSource doesn't change the source

2013-10-22 Thread google-guice
Comment #4 on issue 776 by salman...@google.com: Avoid unnecessary object creation if .withSource doesn't change the source http://code.google.com/p/google-guice/issues/detail?id=776 Right, I agree that it can improve the current situation. I have looked at the problem a few months ago

Re: Issue 759 in google-guice: Use ASM 4.2 (or 5.0_BETA for Java8 support)

2013-10-22 Thread google-guice
Updates: Summary: Use ASM 4.2 (or 5.0_BETA for Java8 support) Comment #5 on issue 759 by mccu...@gmail.com: Use ASM 4.2 (or 5.0_BETA for Java8 support) http://code.google.com/p/google-guice/issues/detail?id=759 FYI, I'm also experimenting with ASM 5.0_BETA (via patched version

Re: Issue 776 in google-guice: Avoid unnecessary object creation if .withSource doesn't change the source

2013-10-22 Thread google-guice
Comment #5 on issue 776 by sberlin: Avoid unnecessary object creation if .withSource doesn't change the source http://code.google.com/p/google-guice/issues/detail?id=776 If it's a quick/easy win for the SPI case, we might as well submit this patch and then look for more optimizations later

Re: Issue 759 in google-guice: Use ASM 4.2 (or 5.0_BETA for Java8 support)

2013-10-22 Thread google-guice
Comment #6 on issue 759 by sberlin: Use ASM 4.2 (or 5.0_BETA for Java8 support) http://code.google.com/p/google-guice/issues/detail?id=759 I'll take a look at pulling in the cglib changes and cutting a release with them. Probably won't be for a bit though. -- You received this message

Re: Comment on MinimizeMutability in google-guice

2013-10-19 Thread google-guice
Comment by andrey.c...@gmail.com: There's probably a TYPO in constructor name. It should be RealPaymentService, not RealPaymentRequestService. For more information: https://code.google.com/p/google-guice/wiki/MinimizeMutability -- You received this message because you are subscribed

Re: Comment on AppsThatUseGuice in google-guice

2013-10-19 Thread google-guice
Comment by alexandr...@gmail.com: http://shorts.axaih.com uses guice/closure template/clusure library For more information: https://code.google.com/p/google-guice/wiki/AppsThatUseGuice -- You received this message because you are subscribed to the Google Groups google-guice-dev group

Re: Comment on AppsThatUseGuice in google-guice

2013-10-19 Thread google-guice
Comment by alexandr...@gmail.com: http://shorts.axaih.com uses guice/guava/closure template/clusure library For more information: https://code.google.com/p/google-guice/wiki/AppsThatUseGuice -- You received this message because you are subscribed to the Google Groups google-guice-dev group

Re: Comment on AssistedInject in google-guice

2013-10-18 Thread google-guice
time and I cant figure out how to use @AssistedInject. any help? For more information: http://code.google.com/p/google-guice/wiki/AssistedInject -- You received this message because you are subscribed to the Google Groups google-guice-dev group. To unsubscribe from this group and stop receiving

Re: Comment on AssistedInject in google-guice

2013-10-18 Thread google-guice
://code.google.com/p/google-guice/wiki/AssistedInject -- You received this message because you are subscribed to the Google Groups google-guice-dev group. To unsubscribe from this group and stop receiving emails from it, send an email to google-guice-dev+unsubscr...@googlegroups.com. To post

Re: Issue 288 in google-guice: FinalizableReferenceQueue still leaks

2013-10-16 Thread google-guice
Comment #95 on issue 288 by r...@asleson.net: FinalizableReferenceQueue still leaks http://code.google.com/p/google-guice/issues/detail?id=288 So this bug has not been fixed in the Guice 4.0 beta, correct? So is it correct to assume that it won't make it into 4.0 GA? Bummer. -- You

Re: Issue 288 in google-guice: FinalizableReferenceQueue still leaks

2013-10-16 Thread google-guice
Comment #96 on issue 288 by mccu...@gmail.com: FinalizableReferenceQueue still leaks http://code.google.com/p/google-guice/issues/detail?id=288 Guice 4.0-beta embeds Guava 11.0.1, which includes the FinalizableReferenceQueue fix. You may still see a warning in Tomcat about a ThreadLocal

Re: Issue 288 in google-guice: FinalizableReferenceQueue still leaks

2013-10-16 Thread google-guice
Comment #97 on issue 288 by cow...@bbs.darktech.org: FinalizableReferenceQueue still leaks http://code.google.com/p/google-guice/issues/detail?id=288 All of this could have been solved 6 (!!) years ago by simply adding a close() method. See comment #3. Add a close() method now and remove

Re: Issue 288 in google-guice: FinalizableReferenceQueue still leaks

2013-10-16 Thread google-guice
Comment #98 on issue 288 by mccu...@gmail.com: FinalizableReferenceQueue still leaks http://code.google.com/p/google-guice/issues/detail?id=288 There's no need for a close() method now - the weak/soft maps were re-implemented in Guava 10 to avoid the need for the FRQ background thread

Re: Issue 597 in google-guice: Persist Extension: UnitOfWork.begin() throws IllegalStateException when called multiple times

2013-10-16 Thread google-guice
Comment #16 on issue 597 by tae...@gmail.com: Persist Extension: UnitOfWork.begin() throws IllegalStateException when called multiple times http://code.google.com/p/google-guice/issues/detail?id=597 I am seeing the same behavior as indicated by the above link regarding the PersistFilter

Re: Issue 597 in google-guice: Persist Extension: UnitOfWork.begin() throws IllegalStateException when called multiple times

2013-10-16 Thread google-guice
Comment #17 on issue 597 by remi.ban...@gmail.com: Persist Extension: UnitOfWork.begin() throws IllegalStateException when called multiple times http://code.google.com/p/google-guice/issues/detail?id=597 Hi, I have implemented a fix for this issue as already mentioned above. http

Re: Issue 597 in google-guice: Persist Extension: UnitOfWork.begin() throws IllegalStateException when called multiple times

2013-10-16 Thread google-guice
Comment #18 on issue 597 by tae...@gmail.com: Persist Extension: UnitOfWork.begin() throws IllegalStateException when called multiple times http://code.google.com/p/google-guice/issues/detail?id=597 So, that does not seem to be quite what I am seeing. My specific case is as follows

Re: Issue 709 in google-guice: Pull Guava out from jarjar and attach it as a normal dependency.

2013-10-16 Thread google-guice
Comment #2 on issue 709 by mccu...@gmail.com: Pull Guava out from jarjar and attach it as a normal dependency. http://code.google.com/p/google-guice/issues/detail?id=709 Issue 689 has been merged into this issue. -- You received this message because this project is configured to send all

Re: Issue 689 in google-guice: Make Guava an external dependency instead of jarjaring

2013-10-16 Thread google-guice
Updates: Status: Duplicate Mergedinto: 709 Comment #1 on issue 689 by mccu...@gmail.com: Make Guava an external dependency instead of jarjaring http://code.google.com/p/google-guice/issues/detail?id=689 (No comment was entered for this change.) -- You received this message

Re: Issue 769 in google-guice: Guice can not be built against Guava 15.0-rc1

2013-10-16 Thread google-guice
Updates: Status: Duplicate Mergedinto: 760 Comment #3 on issue 769 by mccu...@gmail.com: Guice can not be built against Guava 15.0-rc1 http://code.google.com/p/google-guice/issues/detail?id=769 (No comment was entered for this change.) -- You received this message because

Re: Issue 760 in google-guice: Update internal caches to use Guava's CacheBuilder API

2013-10-16 Thread google-guice
Comment #6 on issue 760 by mccu...@gmail.com: Update internal caches to use Guava's CacheBuilder API http://code.google.com/p/google-guice/issues/detail?id=760 Issue 769 has been merged into this issue. -- You received this message because this project is configured to send all issue

Re: Issue 761 in google-guice: How can i set the classloader for the Guice

2013-10-16 Thread google-guice
Comment #3 on issue 761 by mccu...@gmail.com: How can i set the classloader for the Guice http://code.google.com/p/google-guice/issues/detail?id=761 There appears to be some integration support here: https://github.com/englishtown/vertx-mod-guice -- You received this message because

Re: Issue 681 in google-guice: ComputationException due to StackOverflowError during injection is hard to track down

2013-10-10 Thread google-guice
Updates: Status: WontFix Comment #1 on issue 681 by sberlin: ComputationException due to StackOverflowError during injection is hard to track down http://code.google.com/p/google-guice/issues/detail?id=681 There's unfortunately no way we can insert information into the error

Re: Comment on FrequentlyAskedQuestions in google-guice

2013-10-09 Thread google-guice
Comment by gtmbaner...@gmail.com: How can I deprecate a guice injection? For more information: http://code.google.com/p/google-guice/wiki/FrequentlyAskedQuestions -- You received this message because you are subscribed to the Google Groups google-guice-dev group. To unsubscribe from

Re: Comment on ExternalDocumentation in google-guice

2013-10-08 Thread google-guice
Comment by lars.mic...@gmail.com: Another Guice AOP example: http://www.larsmichael.net/?p=124 For more information: http://code.google.com/p/google-guice/wiki/ExternalDocumentation -- You received this message because you are subscribed to the Google Groups google-guice-dev group

Re: Issue 694 in google-guice: JpaPersistService.end() doesn't call entityManager.remove() in finally block

2013-10-08 Thread google-guice
Comment #3 on issue 694 by m...@lucidfox.org: JpaPersistService.end() doesn't call entityManager.remove() in finally block http://code.google.com/p/google-guice/issues/detail?id=694 Any chance this will be fixed for 4.0? -- You received this message because this project is configured

Re: Issue 652 in google-guice: Execution of a MethodInvocation.proceed() can cause an infinite loop

2013-10-08 Thread google-guice
Comment #6 on issue 652 by taviana...@gmail.com: Execution of a MethodInvocation.proceed() can cause an infinite loop http://code.google.com/p/google-guice/issues/detail?id=652 Any reason the attached patch won't work? It passes all the tests and seems simple enough. Attachments

Re: Issue 652 in google-guice: Execution of a MethodInvocation.proceed() can cause an infinite loop

2013-10-08 Thread google-guice
Comment #7 on issue 652 by sberlin: Execution of a MethodInvocation.proceed() can cause an infinite loop http://code.google.com/p/google-guice/issues/detail?id=652 That's a pretty good workaround. Could you maybe add a new multi-threaded test that would have broken prior to the patch also

Re: Issue 652 in google-guice: Execution of a MethodInvocation.proceed() can cause an infinite loop

2013-10-08 Thread google-guice
Comment #8 on issue 652 by taviana...@gmail.com: Execution of a MethodInvocation.proceed() can cause an infinite loop http://code.google.com/p/google-guice/issues/detail?id=652 Sure, attached. The test isn't multi-threaded for simplicity, but it captures the root of the problem

Re: Issue 652 in google-guice: Execution of a MethodInvocation.proceed() can cause an infinite loop

2013-10-08 Thread google-guice
Comment #9 on issue 652 by sberlin: Execution of a MethodInvocation.proceed() can cause an infinite loop http://code.google.com/p/google-guice/issues/detail?id=652 Nice, this LGTM, I'll patch it and test against our corpus and submit if there are no errors. Thanks for submitting

Re: Issue 266 in google-guice: Expose UniqueAnnotations.create()

2013-10-08 Thread google-guice
Comment #2 on issue 266 by taviana...@gmail.com: Expose UniqueAnnotations.create() http://code.google.com/p/google-guice/issues/detail?id=266 Right now I'm re-implementing UniqueAnnotations for my own extension. Basically I have an AbstractMagicModule with a magic() method

Issue 774 in google-guice: Modules.override() loses source information when the current source is skipped

2013-10-07 Thread google-guice
Status: New Owner: New issue 774 by taviana...@gmail.com: Modules.override() loses source information when the current source is skipped http://code.google.com/p/google-guice/issues/detail?id=774 This test case: code language=java Guice.createInjector(new AbstractModule

Issue 773 in google-guice: Patch for /ExternalDocumentation.wiki

2013-10-05 Thread google-guice
Status: New Owner: Labels: Type-Patch New issue 773 by hussainp...@gmail.com: Patch for /ExternalDocumentation.wiki http://code.google.com/p/google-guice/issues/detail?id=773 Learning Google Guice is a latest book published by PackT Publishers in Sept. 2013. It is one of the few books

[google-guice] 0 new revisions pushed by cgru...@google.com on 2013-10-05 20:58 GMT

2013-10-05 Thread google-guice
master moved from ecfa3c2cdcae to 401a6a43b41a 0 new revisions http://code.google.com/p/google-guice/source/list -- You received this message because you are subscribed to the Google Groups google-guice-dev group. To unsubscribe from this group and stop receiving emails from it, send an email

Re: Comment on SpringComparison in google-guice

2013-10-03 Thread google-guice
Comment by rmene...@netflix.com: Erick Burke's link on ported a Spring application to Guice: is currently dead. Goes to the domain hoster's ad page. Please update it. For more information: https://code.google.com/p/google-guice/wiki/SpringComparison -- You received this message because you

Re: Comment on GettingStarted in google-guice

2013-10-03 Thread google-guice
the RealBillingService constructor (not the entire RealBillingService class). Why does this work, and what are the differences between these two approaches? For more information: https://code.google.com/p/google-guice/wiki/GettingStarted -- You received this message because you are subscribed

Re: Comment on GettingStarted in google-guice

2013-10-03 Thread google-guice
Comment by rob.joha...@gmail.com: Looks like I found an answer to my own question on the [https://code.google.com/p/google-guice/wiki/JustInTimeBindings JustInTimeBindings page]: If a type is needed but there isn't an explicit binding, the injector will attempt to create a Just-In-Time

Re: Issue 369 in google-guice: Convenient way to expose multibindings and mapbindings from PrivateModules

2013-10-01 Thread google-guice
Comment #11 on issue 369 by ash...@gmail.com: Convenient way to expose multibindings and mapbindings from PrivateModules http://code.google.com/p/google-guice/issues/detail?id=369 I need this too. I'm using workaround with unique annotation for private multibinder now. -- You received

Re: Comment on ProviderBindings in google-guice

2013-10-01 Thread google-guice
Comment by sebastie...@gmail.com: I think the get method of a provider should respect the @Singleton annotation, and it seems to not respect it in version 3. For more information: https://code.google.com/p/google-guice/wiki/ProviderBindings -- You received this message because you

Re: Comment on ExternalDocumentation in google-guice

2013-09-30 Thread google-guice
Comment by hussainp...@gmail.com: I have written a beginner level book about Guice. Learning Google Guice (http://www.packtpub.com/learning-google-guice/book) published by PackT publishers explores Guice 3.0 and various extensions that surround it. The book beings with general discussions

Re: Comment on AvoidConditionalLogicInModules in google-guice

2013-09-27 Thread google-guice
. For more information: https://code.google.com/p/google-guice/wiki/AvoidConditionalLogicInModules -- You received this message because you are subscribed to the Google Groups google-guice-dev group. To unsubscribe from this group and stop receiving emails from it, send an email to google-guice

Re: Comment on OptionalAOP in google-guice

2013-09-26 Thread google-guice
Comment by cgru...@google.com: Guice 4.0-beta is in the maven central repository, as Stuart pointed out above. Just use the normal maven dependency declaration. For more information: https://code.google.com/p/google-guice/wiki/OptionalAOP -- You received this message because you

[google-guice.wiki] push by - Edited wiki page OptionalAOP through web user interface. on 2013-09-26 15:47 GMT

2013-09-26 Thread google-guice
Revision: 3dbbb97748bd Author: cgruber cgru...@google.com Date: Thu Sep 26 15:47:08 2013 UTC Log: Edited wiki page OptionalAOP through web user interface. http://code.google.com/p/google-guice/source/detail?r=3dbbb97748bdrepo=wiki Modified: /OptionalAOP.wiki

Re: Comment on AppsThatUseGuice in google-guice

2013-09-25 Thread google-guice
Comment by lars.mic...@gmail.com: I used Guice and Jersey to build a RESTful service serving http://waterdata.dhigroup.com/mantaray/home. Blogged about it on http://www.larsmichael.net/?p=114. For more information: https://code.google.com/p/google-guice/wiki/AppsThatUseGuice -- You received

Re: Comment on OptionalAOP in google-guice

2013-09-25 Thread google-guice
Comment by roberto.andrade: Wondering which Maven repo 4.0 (beta) can be fetched from? For more information: http://code.google.com/p/google-guice/wiki/OptionalAOP -- You received this message because you are subscribed to the Google Groups google-guice-dev group. To unsubscribe from

Issue 771 in google-guice: Annotations.isAllDefaultMethods returns false for method less interfaces.

2013-09-24 Thread google-guice
Status: New Owner: New issue 771 by henn...@schmiedehausen.org: Annotations.isAllDefaultMethods returns false for method less interfaces. http://code.google.com/p/google-guice/issues/detail?id=771 Description of the issue: Steps to reproduce: 1. Annotations.isAllDefaultMethods

Re: Issue 771 in google-guice: Annotations.isAllDefaultMethods returns false for method less interfaces.

2013-09-24 Thread google-guice
Comment #1 on issue 771 by sberlin: Annotations.isAllDefaultMethods returns false for method less interfaces. http://code.google.com/p/google-guice/issues/detail?id=771 Is this causing a bug? The method is in inject/internal, and AFAIK it's working as it's expected to work. It returns

Re: Issue 771 in google-guice: Annotations.isAllDefaultMethods returns false for method less interfaces.

2013-09-24 Thread google-guice
Comment #2 on issue 771 by henn...@schmiedehausen.org: Annotations.isAllDefaultMethods returns false for method less interfaces. http://code.google.com/p/google-guice/issues/detail?id=771 This annotation returns false: public @interface Foo { } This annotation returns true: public

Re: Issue 771 in google-guice: Annotations.isAllDefaultMethods returns false for method less interfaces.

2013-09-24 Thread google-guice
Comment #3 on issue 771 by henn...@schmiedehausen.org: Annotations.isAllDefaultMethods returns false for method less interfaces. http://code.google.com/p/google-guice/issues/detail?id=771 And yes, it only returns true if all declared methods have a default value set. The question is, what

Re: Issue 771 in google-guice: Annotations.isAllDefaultMethods returns false for method less interfaces.

2013-09-24 Thread google-guice
Comment #4 on issue 771 by sberlin: Annotations.isAllDefaultMethods returns false for method less interfaces. http://code.google.com/p/google-guice/issues/detail?id=771 What bug is this causing? The code is in internal and is intended for internal use in Guice. External code should

Re: Issue 771 in google-guice: Annotations.isAllDefaultMethods returns false for method less interfaces.

2013-09-24 Thread google-guice
Comment #8 on issue 771 by henn...@schmiedehausen.org: Annotations.isAllDefaultMethods returns false for method less interfaces. http://code.google.com/p/google-guice/issues/detail?id=771 So, the problem is that any piece of code has to drag around this dichtonomy between Annotation (IAW

Re: Issue 771 in google-guice: Annotations.isAllDefaultMethods returns false for method less interfaces.

2013-09-24 Thread google-guice
Comment #9 on issue 771 by mccu...@gmail.com: Annotations.isAllDefaultMethods returns false for method less interfaces. http://code.google.com/p/google-guice/issues/detail?id=771 I have a key that represents a type and either an Annotation or a Class? extends Annotation I want to create

Re: Issue 771 in google-guice: Annotations.isAllDefaultMethods returns false for method less interfaces.

2013-09-24 Thread google-guice
Comment #10 on issue 771 by sberlin: Annotations.isAllDefaultMethods returns false for method less interfaces. http://code.google.com/p/google-guice/issues/detail?id=771 I don't buy that it's a bug. The code is doing exactly what it was written to do. You can argue that the current API

Re: Issue 771 in google-guice: Annotations.isAllDefaultMethods returns false for method less interfaces.

2013-09-24 Thread google-guice
Comment #11 on issue 771 by henn...@schmiedehausen.org: Annotations.isAllDefaultMethods returns false for method less interfaces. http://code.google.com/p/google-guice/issues/detail?id=771 Guess, I read the explanation for ofType wrong. Thanks for pointing this out, I will try

Re: Issue 320 in google-guice: Need a LibraryModule that doesn't inherit from base environment by default

2013-09-22 Thread google-guice
Comment #2 on issue 320 by taviana...@gmail.com: Need a LibraryModule that doesn't inherit from base environment by default http://code.google.com/p/google-guice/issues/detail?id=320 Any plans to implement this? Or can this already be done? -- You received this message because this project

Re: Issue 756 in google-guice: Memory leak by using child injector and multibinder

2013-09-21 Thread google-guice
Comment #1 on issue 756 by johannes...@gmail.com: Memory leak by using child injector and multibinder http://code.google.com/p/google-guice/issues/detail?id=756 Basically what is the problem, is that by using the toString method to put the binding in the blacklist, this will lead to a new

[google-guice] push by cgru...@google.com - Convert from MapMaker to CacheBuilder. Change mostly from Stewart McC... on 2013-09-20 18:39 GMT

2013-09-20 Thread google-guice
flakage in a functional test. - Created by MOE: http://code.google.com/p/moe-java MOE_MIGRATED_REVID=52590552 http://code.google.com/p/google-guice/source/detail?r=0658cc19578a Modified: /core/pom.xml /core/src/com/google/inject/internal/Annotations.java /core/src/com/google/inject

Comment on revision r0658cc19578a8f5b9320adbbb034d98c7fdc6c7b in google-guice

2013-09-20 Thread google-guice
Comment by jdoneill...@gmail.com: General Comment: How do you generate the guice-1.0.xml file for use with the -since param? For more information: https://code.google.com/p/google-guice/source/detail?r=0658cc19578a8f5b9320adbbb034d98c7fdc6c7b -- You received this message because you

[google-guice] 0 new revisions pushed by cgru...@google.com on 2013-09-20 23:45 GMT

2013-09-20 Thread google-guice
master moved from 0658cc19578a to c01ba1d84d54 0 new revisions http://code.google.com/p/google-guice/source/list -- You received this message because you are subscribed to the Google Groups google-guice-dev group. To unsubscribe from this group and stop receiving emails from it, send an email

[google-guice] push by cgru...@google.com - Convert from MapMaker to CacheBuilder. Change mostly from Stewart McC... on 2013-09-21 00:30 GMT

2013-09-20 Thread google-guice
flakage in a functional test. - Created by MOE: http://code.google.com/p/moe-java MOE_MIGRATED_REVID=52590552 http://code.google.com/p/google-guice/source/detail?r=8743a0bb0fb4 Modified: /core/src/com/google/inject/internal/Annotations.java /core/src/com/google/inject/internal

[google-guice.wiki] push by - Add API docs link. on 2013-09-21 00:44 GMT

2013-09-20 Thread google-guice
Revision: 5093ad45d726 Author: cgruber cgru...@google.com Date: Sat Sep 21 00:44:43 2013 UTC Log: Add API docs link. http://code.google.com/p/google-guice/source/detail?r=5093ad45d726repo=wiki Modified: /Guice40.wiki === --- /Guice40.wiki

[google-guice.wiki] push by - Add in API differential documentation. on 2013-09-21 00:49 GMT

2013-09-20 Thread google-guice
Revision: 6d64f80630cb Author: cgruber cgru...@google.com Date: Sat Sep 21 00:49:30 2013 UTC Log: Add in API differential documentation. http://code.google.com/p/google-guice/source/detail?r=6d64f80630cbrepo=wiki Modified: /Guice40.wiki

Re: Comment on Guice40 in google-guice

2013-09-20 Thread google-guice
: https://code.google.com/p/google-guice/wiki/Guice40 -- You received this message because you are subscribed to the Google Groups google-guice-dev group. To unsubscribe from this group and stop receiving emails from it, send an email to google-guice-dev+unsubscr...@googlegroups.com. To post

Re: Comment on MinimizeMutability in google-guice

2013-09-17 Thread google-guice
Comment by qiang...@gmail.com: Refactoring may cause headache if going with constructor injection? e.g adding more fields. You may end up with a constructor with many parameters... For more information: https://code.google.com/p/google-guice/wiki/MinimizeMutability -- You received

Re: Issue 369 in google-guice: Convenient way to expose multibindings and mapbindings from PrivateModules

2013-09-17 Thread google-guice
Comment #10 on issue 369 by justindh...@gmail.com: Convenient way to expose multibindings and mapbindings from PrivateModules http://code.google.com/p/google-guice/issues/detail?id=369 My solution is to use expose in the configure method of the PrivateModule with the help of TypeLiteral

Re: Issue 251 in google-guice: @ProvidedBy @Singleton doesn't work

2013-09-17 Thread google-guice
Comment #2 on issue 251 by ben.ma...@gmail.com: @ProvidedBy @Singleton doesn't work http://code.google.com/p/google-guice/issues/detail?id=251 Any chance this is fixed in Guice 4.0? Just ran into this for a servlet filter. -- You received this message because this project is configured

Re: Issue 530 in google-guice: Add FactoryModuleBuilder.getProvider to support Command Pattern use case

2013-09-16 Thread google-guice
Comment #3 on issue 530 by jo...@weightpoint.se: Add FactoryModuleBuilder.getProvider to support Command Pattern use case http://code.google.com/p/google-guice/issues/detail?id=530 Is this still relevant? Trying to do the same thing but the use case seems outdated -- You received

Re: Issue 760 in google-guice: Update internal caches to use Guava's CacheBuilder API

2013-09-12 Thread google-guice
Comment #3 on issue 760 by mccu...@gmail.com: Update internal caches to use Guava's CacheBuilder API http://code.google.com/p/google-guice/issues/detail?id=760 Latest patch against master Attachments: GUICE_760_switch_to_cachebuilder.patch 12.9 KB -- You received this message

Re: Issue 760 in google-guice: Update internal caches to use Guava's CacheBuilder API

2013-09-12 Thread google-guice
Updates: Status: Accepted Owner: cgru...@google.com Labels: Type-CodeQuality Milestone-Release4.0 Comment #4 on issue 760 by cgru...@google.com: Update internal caches to use Guava's CacheBuilder API http://code.google.com/p/google-guice/issues/detail?id=760

Re: Issue 760 in google-guice: Update internal caches to use Guava's CacheBuilder API

2013-09-12 Thread google-guice
Comment #5 on issue 760 by cgru...@google.com: Update internal caches to use Guava's CacheBuilder API http://code.google.com/p/google-guice/issues/detail?id=760 Thanks Stuart. I'll try to get this in this week. -- You received this message because this project is configured to send all

Issue 769 in google-guice: Guice can not be built against Guava 15.0-rc1

2013-09-06 Thread google-guice
Status: New Owner: New issue 769 by st...@entreesystems.com: Guice can not be built against Guava 15.0-rc1 http://code.google.com/p/google-guice/issues/detail?id=769 com.google.inject.internal.Annotations uses the deprecated MapMaker.makeComputingMap() method which, in this latest

[google-guice.wiki] push by - Edited wiki page ExternalDocumentation through web user interface. on 2013-09-03 17:07 GMT

2013-09-03 Thread google-guice
Revision: 9a9e69672f18 Author: sberlin sber...@gmail.com Date: Tue Sep 3 17:07:44 2013 UTC Log: Edited wiki page ExternalDocumentation through web user interface. http://code.google.com/p/google-guice/source/detail?r=9a9e69672f18repo=wiki Modified: /ExternalDocumentation.wiki

Re: Comment on Guice40 in google-guice

2013-09-03 Thread google-guice
Comment by alexandr...@gmail.com: Guice 4 supports async servlet? https://code.google.com/p/google-guice/issues/detail?id=650 For more information: https://code.google.com/p/google-guice/wiki/Guice40 -- You received this message because you are subscribed to the Google Groups google-guice

Re: Comment on GuiceDiscussions in google-guice

2013-09-02 Thread google-guice
Comment by alexandr...@gmail.com: Guice 4 supports async servlet? https://code.google.com/p/google-guice/issues/detail?id=650 For more information: https://code.google.com/p/google-guice/wiki/GuiceDiscussions -- You received this message because you are subscribed to the Google Groups

Re: Comment on Guice40 in google-guice

2013-09-02 Thread google-guice
Comment by liuyuan...@gmail.com: The worst thing in 3.0 is error message. If any thing go wrong in initialization, it generated a mass of meaningless message. I hope 4.0 have improved this For more information: https://code.google.com/p/google-guice/wiki/Guice40 -- You received

Issue 768 in google-guice: guice-servlet+guice-persist -- Memory leak

2013-08-28 Thread google-guice
Status: New Owner: New issue 768 by m.zabel...@gmail.com: guice-servlet+guice-persist -- Memory leak http://code.google.com/p/google-guice/issues/detail?id=768 When a simple guice servlet application is created, e.g. following the example in the wiki (https://code.google.com/p

Re: Comment on InjectingProviders in google-guice

2013-08-27 Thread google-guice
= inj.getInstance(Bar.class); Foo fooOne = myBar.getFoo(); Foo fooTwo = myBar.getFoo(); you can also take on this blog site: http://nfljerseyscheapjerseyscheapnfljerseys.blogspot.com For more information: http://code.google.com/p/google-guice/wiki/InjectingProviders

Re: Comment on InjectionPoints in google-guice

2013-08-22 Thread google-guice
://code.google.com/p/google-guice/wiki/InjectionPoints -- You received this message because you are subscribed to the Google Groups google-guice-dev group. To unsubscribe from this group and stop receiving emails from it, send an email to google-guice-dev+unsubscr...@googlegroups.com. To post to this group

Re: Comment on InjectionPoints in google-guice

2013-08-22 Thread google-guice
://code.google.com/p/google-guice/wiki/InjectionPoints -- You received this message because you are subscribed to the Google Groups google-guice-dev group. To unsubscribe from this group and stop receiving emails from it, send an email to google-guice-dev+unsubscr...@googlegroups.com. To post to this group

Re: Comment on InjectionPoints in google-guice

2013-08-22 Thread google-guice
://code.google.com/p/google-guice/wiki/InjectionPoints -- You received this message because you are subscribed to the Google Groups google-guice-dev group. To unsubscribe from this group and stop receiving emails from it, send an email to google-guice-dev+unsubscr...@googlegroups.com. To post

Issue 767 in google-guice: Provide all versions of Guice in Maven Central Repository....only Guice v1.0 is available right now!

2013-08-22 Thread google-guice
Status: New Owner: New issue 767 by colbert.philippe: Provide all versions of Guice in Maven Central Repositoryonly Guice v1.0 is available right now! http://code.google.com/p/google-guice/issues/detail?id=767 Description of the issue: Steps to reproduce: 1. Search Maven

Re: Issue 730 in google-guice: Automatically started UnitOfWork is never ended

2013-08-22 Thread google-guice
Comment #4 on issue 730 by bimsc...@gmail.com: Automatically started UnitOfWork is never ended http://code.google.com/p/google-guice/issues/detail?id=730 Will this fix be part of Guice 4.0? -- You received this message because this project is configured to send all issue notifications

Re: Issue 767 in google-guice: Provide all versions of Guice in Maven Central Repository....only Guice v1.0 is available right now!

2013-08-22 Thread google-guice
Updates: Status: Invalid Comment #1 on issue 767 by mccu...@gmail.com: Provide all versions of Guice in Maven Central Repositoryonly Guice v1.0 is available right now! http://code.google.com/p/google-guice/issues/detail?id=767 All Guice versions from 1.0 to 4.0-beta are already

Re: Issue 730 in google-guice: Automatically started UnitOfWork is never ended

2013-08-22 Thread google-guice
Comment #6 on issue 730 by spootsy@gmail.com: Automatically started UnitOfWork is never ended http://code.google.com/p/google-guice/issues/detail?id=730 I was hoping they'd include a patch of some kind but I don't see any changes in trunk. It's a shame, because the current semantics

Re: Comment on Guice40 in google-guice

2013-08-21 Thread google-guice
Comment by gem...@gmail.com: when 4.0 will be released? For more information: https://code.google.com/p/google-guice/wiki/Guice40 -- You received this message because you are subscribed to the Google Groups google-guice-dev group. To unsubscribe from this group and stop receiving emails from

Re: Issue 766 in google-guice: Christian Gruber Blog Roll Link

2013-08-20 Thread google-guice
Updates: Status: Fixed Comment #1 on issue 766 by christia...@gmail.com: Christian Gruber Blog Roll Link http://code.google.com/p/google-guice/issues/detail?id=766 Heh. Thanks. I neglected to migrate this when I moved to the US. :) Clearly I need to blog more. -- You received

Issue 764 in google-guice: @Finder annotation issue

2013-08-19 Thread google-guice
Status: New Owner: New issue 764 by vesko.m@gmail.com: @Finder annotation issue http://code.google.com/p/google-guice/issues/detail?id=764 In @Finder annotation returnAs() has default value Collection.class. However, if you forget to specify it, it throws an exception because

Issue 765 in google-guice: Lazily loaded collections has no session if method is not marked @Transactional

2013-08-19 Thread google-guice
Status: New Owner: New issue 765 by alex.che...@gmail.com: Lazily loaded collections has no session if method is not marked @Transactional http://code.google.com/p/google-guice/issues/detail?id=765 Description of the issue: Encountered a problem with how lazy loaded collections were

Issue 766 in google-guice: Christian Gruber Blog Roll Link

2013-08-19 Thread google-guice
Status: New Owner: New issue 766 by robotta...@gmail.com: Christian Gruber Blog Roll Link http://code.google.com/p/google-guice/issues/detail?id=766 Description of the issue: Steps to reproduce: 1. Click Link 2. See domain squatting page. -- You received this message because

Re: Issue 382 in google-guice: No source jars in maven repository

2013-08-18 Thread google-guice
Comment #13 on issue 382 by leetch...@gmail.com: No source jars in maven repository http://code.google.com/p/google-guice/issues/detail?id=382 anything for guice 2.0 backports? thanks for the sources-jar but would be nice to have it in mvn repo too! -- You received this message because

Re: Comment on Guice40 in google-guice

2013-08-17 Thread google-guice
Comment by patrick@gmail.com: You can easily create the jdiff yourself: {{{git clone https://code.google.com/p/google-guice/}}} modify build.xml: {{{ property name=old.api value=3.0/ property name=new.api value=4.0/ }}} run the ant target: {{{ant jdiff}}} For more information: https

Re: Issue 618 in google-guice: Make guice-servlet more adaptable

2013-08-15 Thread google-guice
Comment #14 on issue 618 by queen...@arcbees.com: Make guice-servlet more adaptable http://code.google.com/p/google-guice/issues/detail?id=618 Hasn't this been fixed? -- You received this message because this project is configured to send all issue notifications to this address. You may

Issue 762 in google-guice: Logger should be injected into static members

2013-08-14 Thread google-guice
Status: New Owner: New issue 762 by mwisni...@gmail.com: Logger should be injected into static members http://code.google.com/p/google-guice/issues/detail?id=762 While guice has built-in support for injecting java.util.logging.Logger into instance fields, it does no such thing

Issue 763 in google-guice: something is weird, using thread pool in AOP

2013-08-14 Thread google-guice
Status: New Owner: New issue 763 by chenwei...@gmail.com: something is weird, using thread pool in AOP http://code.google.com/p/google-guice/issues/detail?id=763 Description of the issue: using thread pool in AOP: fixedPool.submit there is weird that fixedPool.submit called a lots times

Re: Issue 763 in google-guice: something is weird, using thread pool in AOP

2013-08-14 Thread google-guice
Comment #1 on issue 763 by chenwei...@gmail.com: something is weird, using thread pool in AOP http://code.google.com/p/google-guice/issues/detail?id=763 Description of the issue: using thread pool in AOP: fixedPool.submit there is weird that fixedPool.submit called a lots times Steps

Re: Issue 763 in google-guice: something is weird, using thread pool in AOP

2013-08-14 Thread google-guice
Updates: Status: WorkingAsIntended Comment #2 on issue 763 by sberlin: something is weird, using thread pool in AOP http://code.google.com/p/google-guice/issues/detail?id=763 AOP in Guice does not allow offloading the invocation to a separate thread. -- You received this message

Re: Issue 218 in google-guice: Assisted inject can't match generics parameters correctly

2013-08-14 Thread google-guice
Comment #14 on issue 218 by ssk...@qualys.com: Assisted inject can't match generics parameters correctly http://code.google.com/p/google-guice/issues/detail?id=218 I am also facing a situation similar to @ogregoire. Something to the effect of: class AugmentedQueryFactory { public X

Re: Comment on 3rdPartyModules in google-guice

2013-08-12 Thread google-guice
Comment by andrei.w...@gmail.com: jcatapult site: https://code.google.com/p/jcatapult/ For more information: https://code.google.com/p/google-guice/wiki/3rdPartyModules -- You received this message because you are subscribed to the Google Groups google-guice-dev group. To unsubscribe from

Re: Issue 761 in google-guice: How can i set the classloader for the Guice

2013-08-11 Thread google-guice
Comment #1 on issue 761 by mccu...@gmail.com: How can i set the classloader for the Guice http://code.google.com/p/google-guice/issues/detail?id=761 Could you give more details about the particular issue you're trying to solve? Guice operates on types not XML, and types already have

Re: Issue 761 in google-guice: How can i set the classloader for the Guice

2013-08-11 Thread google-guice
Comment #2 on issue 761 by stream1...@gmail.com: How can i set the classloader for the Guice http://code.google.com/p/google-guice/issues/detail?id=761 Well, i want to use guice in vertx (http://vertx.io.) as lang extend, but a lib. Vertx have own classloader, i should use that class

<    1   2   3   4   5   6   7   8   9   10   >