[hibernate-dev] IntelliJ Headers

Emmanuel Bernard emmanuel at hibernate.org
Fri Jul 18 10:18:00 EDT 2008


BTW the intelliJ codestyle is here
http://www.hibernate.org/438.html#A4

--
Emmanuel Bernard
http://in.relation.to/Bloggers/Emmanuel | http://blog.emmanuelbernard.com 
  | http://twitter.com/emmanuelbernard
Hibernate Search in Action (http://is.gd/Dl1)

On  Jul 17, 2008, at 10:58, Sanne Grinovero wrote:

> Hi,
> I'm using eclipse and checking my style manually, as I don't have  
> the template
> and wouldn't trust myself in automatically reformatting my code with  
> a self-made template.
> Who can provide it? Should I try making it myself?
>
> Also I could commit several missing "propset svn:keywords "Id"" and  
> the keyword,
> but stopped myself to ask your permission first as they are many  
> files.
>
> thanks for all information about trunk.
>
> Sanne
>
> 2008/7/16 Emmanuel Bernard <emmanuel at hibernate.org>:
> I've (re-)fixed the conflict of patches we had between Navin's  
> commit and my original proposed fixes. I've also fixed a but between  
> annotations head and search head wrt event listener initializations  
> (that's why I don't like ivy not pointing to latest :) )
>
> Please please please everyone:
>  - use the right style template. I have one for IntelliJ I can  
> provide, I'm sure someone has one for Eclipse. No more style  
> conflict madness.
>  - run ant clean junit before committing. If it fails, do not commit.
>
>
> BTW We are close to the beta 1 freeze, likely to be tonight or  
> tomorrow
>
> A few questions and comments
> 	@ProvidedId
> Should @ProvidedId be renamed @ExternalId?
> ProvidedId.bridge() is no longer of type Class<StringBridge>, just  
> Class (see later)
> name() defaults to providedId (I still think "id" is a better name,  
> though)
>
> 	SearchFactoryImpl
> I've done various refactorings to eliminate the dependency between  
> o.h.c.Configuration and SearchFactoryImpl
> The event system is now responsible for converting the  
> o.h.c.Configuration into a Search configuration
>
> 	TransactionContext
> I moved them next to the Worker and Worker queues in backend (its  
> implementation is in backend.impl
> renamed isTxInProgress() to a readable name
> I removed TransactionContext as Serializable, each implementation  
> shold make its own decision. EventsourceTC is not serialzible: they  
> could use transient / readObject / writeObject
>
> 	SearchConfiguration
> The HibernameImplementation now deal with getMappedClass() returning  
> null. I used Class<?> instead of the raw Class in the interface.
>
>
>
>
> Here are the things Navin still needs to work on:
> 	DocumentBuilder
> 1. @ProvidedId can be set on a superclass, your code does not deal  
> with it
> 2. you've hardcoded the bridge implementation to StringBridge, this  
> is not good and should be configured by ProvidedId.bridge().This  
> should be done more or less like  
> BridgeFactory.extractType(ClassBridge);
>
> Those 2 need to be fixed before we go and release the @ProvidedId  
> feature but this can wait till after Beta1
>
>
>
>
> --
> Emmanuel Bernard
> http://in.relation.to/Bloggers/Emmanuel | http://blog.emmanuelbernard.com 
>  | http://twitter.com/emmanuelbernard
> Hibernate Search in Action (http://is.gd/Dl1)
>
>
> _______________________________________________
> hibernate-dev mailing list
> hibernate-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/hibernate-dev
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/hibernate-dev/attachments/20080718/b57d28f5/attachment.html 


More information about the hibernate-dev mailing list