gen_server2 icon indicating copy to clipboard operation
gen_server2 copied to clipboard

specs of gen_server2

Open kostis opened this issue 14 years ago • 1 comments

I was trying to track the origin of gen_server2 and I came to this place. Apologies if it's not the real origin.

I noticed that old fashioned specs are still used in the code:

-spec(hibernate/1 :: (gs2_state()) -> no_return()).

You may want to modernize these and make them look as follows:

-spec hibernate(gs2_state()) -> no_return().

Also, perhaps the use_specs ifdef is not needed anymore.

kostis avatar May 26 '11 13:05 kostis

Hi Kostis - thanks for pointing this out. The gen_server2 code is synced from the rabbitmq mercurial repository. I've been discussing with them to maintain the gen_server2 code as a separate repo or to get it fully contributed to OTP. In the meanwhile, I'll patch the -specs and submit back to their repo and see if they're willing to accept the change for now.

Cheers!

hyperthunk avatar May 26 '11 15:05 hyperthunk