[erlang-patches] Add spec for base64:encode_binary/1

Kostis Sagonas kostis@REDACTED
Wed Aug 18 13:26:25 CEST 2010


Paul Oliver wrote:
> Dialyzer was reporting that use of base64:encode(Foo :: binary()) had spec
> binary() -> <<_:_*1>>.  The binary comprehension in base64:encode seems to
> result in the return type of <<_:_*1>>.  The patch below enforces a return
> type of binary().

Sorry, but I cannot understand your mail... Can you elaborate a bit?

First of all, dialyzer cannot be reporting what you are claiming because 
the spec of base64:encode/1 is:

   -spec encode(string() | binary()) -> binary().

You are probably referring to some other function...

Looking at your patch, it seems that it refers to base64:encode_binary/1 
which is a *module-local* function, not an exported one. Why do you feel 
the need to add a spec to this function? What exactly is it that changes 
in your use case?


Kostis


More information about the erlang-patches mailing list