Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

option io_server can also be an atom() #80

Merged
merged 1 commit into from
Jun 4, 2020

Conversation

gomoripeti
Copy link
Contributor

(not for dialyzer but human consumption)

Very minor thing: it happened to me that when I quickly looked at the option section of the docs online (http://ferd.github.io/recon/recon_trace.html#type-options) I initially thought that io_server option must be a pid (maybe recon does something special with it). I only later saw that the textual docs clearly state that it also can be an atom and looking at the code also makes this obvious.

@ferd
Copy link
Owner

ferd commented Jun 4, 2020

Yeah, as long as the atom refers to a registered process, it is allowed.

@ferd ferd merged commit 46b85c0 into ferd:master Jun 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants