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

Try to clarify wording #156

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Try to clarify wording #156

wants to merge 5 commits into from

Conversation

mlimber
Copy link
Contributor

@mlimber mlimber commented Mar 1, 2018

I found the phrasing a little hard to understand and had to read it several times to get it. I've tried to rephrase from what I "got".

I found the phrasing a little hard to understand and had to read it several times to get it. I've tried to rephrase from what I "got".
@tituswinters
Copy link
Contributor

Thanks for the patch - we'll get back to you soon-ish. Kind of a busy week just now, might wind up being Monday.

@mlimber
Copy link
Contributor Author

mlimber commented Mar 1, 2018

No rush on my side. Thanks for caring. :-)

The main gripe I have about the revised text is the "is" vs. "has" notion, which is what I interpreted the original text to be getting at. The "has" language was floating behind the original text, and I have made it explicit (as it were) in my PR.

Construction by conversion, however, does not imply an "is" relationship in the static/dynamic polymorphic sense. To me "is" is too strong a term and is really overloading the idea of polymorphism (which is a little meta). For instance, converting a float to an int is a valid conversion, but while both are conceptually numbers, an int is not a float and there may be something lost (viz., precision, range, NaNishness, etc.).

Perhaps the distinction I'm looking for is better communicated as being between converting contructors and building constructors. The former can be implicit, while the latter should be explicit.

@tituswinters
Copy link
Contributor

tituswinters commented Mar 1, 2018 via email

Copy link
Contributor

@manshreck manshreck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agree that "is" vs. "has" is problematic here. It's a loaded term in programming and it might lead people to think of polymorphism, etc. On the face of it, I think the clarification in the first paragraph is a good qualification, but I'm less happy with the second paragraph for those reasons stated above.

I think more accurately it's a uniqueness statement (do these parameters uniquely identify the constructed object), or perhaps a phrasing with respect to degrees of freedom would be better. The Request class has an implicit "state" of "direction" in addition to server, connection that makes it specifically a Request, rather than a Response, for example.

@mlimber
Copy link
Contributor Author

mlimber commented Mar 1, 2018

I'll take another whack at it.

@mlimber
Copy link
Contributor Author

mlimber commented Jul 13, 2018

Ping! Just noticed this never went anywhere. No worries if you reject.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants