[seam-dev] Seam 2.1.1.CR2

[Sanne Grinovero]

2008/12/4 Pete Muir <pmuir at redhat.com>:
>
> On 4 Dec 2008, at 16:57, Francisco Jose Peredo wrote:
>
>> Hi!
>>
>> Great job Dan!
>>
>> You missed
>>
>> http://www.seamframework.org/Community/PossibleBugInOrgjbossseamcoreValidators
>> ;-)
>
> -> JIRA as Dan and I said
>
>>
>>
>> Oh, and what is you opinon on:
>>
>>
>> http://www.seamframework.org/Community/IfAMethodReturnsBooleanItShouldntReturnNullGoodOrBadPractice
>> ?
>
> I think its fine, null == undefined which is valid state for something to be
> in.

This is very dangerous, as autoboxing will let you use it as a
primitive "boolean"; client code would not expect a
NullPointerException.
I don't know how you are using it, but this is definitely not a good
practice; for a public API even worse.

>
>>
>>
>> Regards,
>>
>> Francisco
>>
>> Pete Muir escribió:
>>>
>>> Standard rules apply, one JIRA per distinct issue.
>>>
>>> On 4 Dec 2008, at 16:24, Dan Allen wrote:
>>>
>>>> I fixed the ManagedEntityWrapper, JpaPermissionStore, and
>>>> ConverterChain. As for the rest, please submit a general JIRA for
>>>> addressing FindBugs best practices (things like null checks, etc).
>>>> Pete, does that sound like the best approach?
>>>>
>>>> -Dan
>>>>
>>>> On Thu, Dec 4, 2008 at 10:31 AM, Francisco Peredo
>>>> <franciscoperedo at tabasco.gob.mx> wrote:
>>>>>
>>>>> Hi!
>>>>>
>>>>> It would be nice if this bugs could be fixed ;-):
>>>>>
>>>>> http://www.seamframework.org/Community/MorePossibleBugsInManyPlaces
>>>>>
>>>>> http://www.seamframework.org/Community/PossibleBugInOrgjbossseamcoreValidators
>>>>>
>>>>>
>>>>> http://www.seamframework.org/Community/PossibleBugInManagedEntityWrapper
>>>>>
>>>>>
>>>>> http://www.seamframework.org/Community/PossibleBugInEjbEntityDescriptorjava
>>>>>
>>>>>
>>>>> http://www.seamframework.org/Community/PossibleBugInJpaPermissionStorejave
>>>>>
>>>>> http://www.seamframework.org/Community/PossibleConvertertChainjavaBug
>>>>>
>>>>> http://www.seamframework.org/Community/IfAMethodReturnsBooleanItShouldntReturnNullGoodOrBadPractice
>>>>>
>>>>>
>>>>> Sorry for cross-posting, I posted all this in the seam user forum,
>>>>> but now
>>>>> that I think about it they seem to be more "seam dev related" than
>>>>> "seam
>>>>> user related" so now I believe maybe I should have posted them here...
>>>>> Am I right?
>>>>>
>>>>> Regards,
>>>>>
>>>>> Francisco
>>>>>
>>>>>
>>>>> Pete Muir-2 wrote:
>>>>>>
>>>>>> All,
>>>>>>
>>>>>> We are aiming for a code freeze for 2.1.1.CR2 for midday Thursday
>>>>>> (EST), and then move quickly to the GA.
>>>>>>
>>>>>> If you've got anything you want in for 2.1.1.GA, make sure it is in by
>>>>>> the 2.1.1.CR2 code freeze :-)
>>>>>>
>>>>>> Best,
>>>>>>
>>>>>> Pete
>>>>>> _______________________________________________
>>>>>> seam-dev mailing list
>>>>>> seam-dev at lists.jboss.org
>>>>>> https://lists.jboss.org/mailman/listinfo/seam-dev
>>>>>>
>>>>>>
>>>>>
>>>>> --
>>>>> View this message in context:
>>>>> http://www.nabble.com/Seam-2.1.1.CR2-tp20793809p20835746.html
>>>>> Sent from the seam-dev mailing list archive at Nabble.com.
>>>>>
>>>>> _______________________________________________
>>>>> seam-dev mailing list
>>>>> seam-dev at lists.jboss.org
>>>>> https://lists.jboss.org/mailman/listinfo/seam-dev
>>>>>
>>>>
>>>>
>>>>
>>>> --
>>>> Dan Allen
>>>> Software consultant | Author of Seam in Action
>>>>
>>>> http://mojavelinux.com
>>>> http://mojavelinux.com/seaminaction
>>>>
>>>> NOTE: While I make a strong effort to keep up with my email on a daily
>>>> basis, personal or other work matters can sometimes keep me away
>>>> from my email. If you contact me, but don't hear back for more than a
>>>> week,
>>>> it is very likely that I am excessively backlogged or the message was
>>>> caught in the spam filters.  Please don't hesitate to resend a
>>>> message if
>>>> you feel that it did not reach my attention.
>>>> _______________________________________________
>>>> seam-dev mailing list
>>>> seam-dev at lists.jboss.org
>>>> https://lists.jboss.org/mailman/listinfo/seam-dev
>>>
>>>
>>
>>
>> --
>> Dirección Informática de Servicios Financieros
>> Dirección General de Modernización e Innovación Gubernamental
>> Secretaría de Administración y Finanzas
>> Paseo de la Sierra 435 col. Reforma
>> C.P. 86086, Villahermosa, Tabasco.
>> Tel. 52 + 993 + 310 40 00 Ext. 7127
>> http://saf.tabasco.gob.mx/
>>
>> IMPORTANTE: Esta transmisión electrónica, incluyendo sus anexos, archivos
>> insertados o "attachments", puede constituir información confidencial o
>> reservada, en los términos de la Ley de Acceso a la Información Pública del
>> Estado de Tabasco, y estar protegida por el derecho fundamental a la
>> privacidad. Se prohibe el uso de esta información por cualquier persona
>> distinta al receptor intencional o previsto. Si usted ha recibibido esta
>> transmisión electrónica por error, por favor responda inmediatamente al
>> emisor y borre esta información de su sistema. El uso, diseminación,
>> distribución o reproducción de esta transmisión electrónica por receptores
>> no intencionados o no previstos por el emisor, no está autorizada y puede
>> considerarse ilícita en los términos de la legislación penal y civil
>> vigente.
>>
>> _______________________________________________
>> seam-dev mailing list
>> seam-dev at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/seam-dev
>
>
> _______________________________________________
> seam-dev mailing list
> seam-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/seam-dev
>