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

Separate Path and Shape types? #234

Closed
jvoigtlaender opened this Issue May 1, 2015 · 5 comments

Comments

Projects
None yet
3 participants
@jvoigtlaender
Contributor

jvoigtlaender commented May 1, 2015

The documentation gives the impression that

outlined : LineStyle -> Shape -> Form

and

traced : LineStyle -> Path -> Form

have different types. But they don't. Which can lead to "wrong applications", like for this user.

@hoosierEE

This comment has been minimized.

Show comment
Hide comment
@hoosierEE

hoosierEE May 1, 2015

Yeah, I think you're right about this. The compiler should complain if someone gives traced a circle, or outlined a segment. I experienced this myself, twice. Once because I made the mistake, and twice because I forgot about the first time!

hoosierEE commented May 1, 2015

Yeah, I think you're right about this. The compiler should complain if someone gives traced a circle, or outlined a segment. I experienced this myself, twice. Once because I made the mistake, and twice because I forgot about the first time!

@evancz

This comment has been minimized.

Show comment
Hide comment
@evancz

evancz May 1, 2015

Member

I agree, I implemented this in a long long ago time when it wasn't clear whether type aliases would be distinct in some way. In any case, I think it makes sense to hide all the internal details by making both of them a type

Member

evancz commented May 1, 2015

I agree, I implemented this in a long long ago time when it wasn't clear whether type aliases would be distinct in some way. In any case, I think it makes sense to hide all the internal details by making both of them a type

@jvoigtlaender

This comment has been minimized.

Show comment
Hide comment
@jvoigtlaender

jvoigtlaender May 1, 2015

Contributor

Cool, I'll send a PR.

Contributor

jvoigtlaender commented May 1, 2015

Cool, I'll send a PR.

@jvoigtlaender

This comment has been minimized.

Show comment
Hide comment
Contributor

jvoigtlaender commented Jun 30, 2015

@hoosierEE

This comment has been minimized.

Show comment
Hide comment
@hoosierEE

hoosierEE Jun 30, 2015

Nice, thanks!

hoosierEE commented Jun 30, 2015

Nice, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment