[erlang-patches] Fix wrong spec for ets:new/2 in HiPE/dialyzer

Niclas Axelsson burbas@REDACTED
Tue Dec 21 15:50:00 CET 2010


On 12/20/2010 12:05 PM, Kostis Sagonas wrote:
> Jesper Louis Andersen wrote:
>> On Thu, Dec 16, 2010 at 10:20, Kostis Sagonas <kostis@REDACTED> wrote:
>>> Your Erlang code is ok, but please change some things in the comment:
>>>
>>> ...
>>
>>> If you do these changes, your patch can be included as is in 'dev'.
>>
>> I'll let cooking be up to you to decide :) Personally I was aiming for
>> a cooking in 'pu' first - mind you, I have not executed the test cases
>> on it, partially because I don't know what tests to narrow it down to.
>> It would seem easy to include in 'pu' for a batch of testing and then
>> just grad it to 'dev'. But as I said, your call.
>>
>> Ammended patch is refetchable at,
>>
>> git fetch git://github.com/jlouis/otp.git fix-hipe-spec-ets-new
>
> I've tested this patch and it is OK.  It can be included as is in 'dev'.
>
> Kostis
>
> ________________________________________________________________
> erlang-patches (at) erlang.org mailing list.
> See http://www.erlang.org/faq.html
> To unsubscribe; mailto:erlang-patches-unsubscribe@REDACTED

Hi Jesper and Kostis,
The patch took a ride in the 'pu' branch during the night for testing 
purposes.
All went well and the patch is now included in 'dev'.

Regards,
Niclas Axelsson, Erlang/OTP


More information about the erlang-patches mailing list