+1
On 30 Oct 2007, at 09:53, Saravanan Subbiah wrote:
> The formatting got messed up when I sent the mail last time... take
> #2.
>
>
> I like this
>
> public ObjectManagerImpl(ObjectManagerConfig config,
> ThreadGroup gcThreadGroup,
> ClientStateManager stateManager,
> ManagedObjectStore objectStore,
> EvictionPolicy cache,
> PersistenceTransactionProvider
> persistenceTransactionProvider,
> Sink faultSink,
> Sink flushSink) {
>
> better than this
>
>
> public ObjectManagerImpl(ObjectManagerConfig config,
> ThreadGroup gcThreadGroup, ClientStateManager stateManager,
> ManagedObjectStore objectStore, EvictionPolicy cache,
> PersistenceTransactionProvider persistenceTransactionProvider,
> Sink faultSink, Sink flushSink) {
>
>
>
>
>
>
> Saravanan Subbiah wrote:
>>
>>
>> I dont have a strong opinion on this, but I like this
>>
>> public ObjectManagerImpl(ObjectManagerConfig config, ThreadGroup
>> gcThreadGroup, ClientStateManager stateManager,
>> ManagedObjectStore objectStore,
>> EvictionPolicy cache,
>> PersistenceTransactionProvider
>> persistenceTransactionProvider, Sink faultSink, Sink flushSink) {
>>
>>
>> better than this
>>
>> public ObjectManagerImpl(ObjectManagerConfig config, ThreadGroup
>> gcThreadGroup, ClientStateManager stateManager,
>> ManagedObjectStore objectStore, EvictionPolicy
>> cache,PersistenceTransactionProvider persistenceTransactionProvider,
>> Sink faultSink, Sink flushSink) {
>>
>>
>> To me the first one is more readable than the second.
>>
>>
>>
>
> _______________________________________________
> tc-dev mailing list
> [email protected]
> http://lists.terracotta.org/mailman/listinfo/tc-dev
--
Geert Bevin
Terracotta - http://www.terracotta.org
Uwyn "Use what you need" - http://uwyn.com
RIFE Java application framework - http://rifers.org
Music and words - http://gbevin.com
_______________________________________________
tc-dev mailing list
[email protected]
http://lists.terracotta.org/mailman/listinfo/tc-dev
