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

BetterC document. #14

Merged
merged 1 commit into from
Feb 16, 2019
Merged

BetterC document. #14

merged 1 commit into from
Feb 16, 2019

Conversation

TurkeyMan
Copy link
Contributor

No description provided.

@p0nce
Copy link
Contributor

p0nce commented Jan 20, 2019

I think things should be made -betterC afterwards, when needed, and that it shouldn't govern our current efforts. -betterC is pretty drastic in its current form.

@TurkeyMan
Copy link
Contributor Author

PR to amend the text you disagree with. I just want to get the state of the conversation recorded, since it seems contentious.

@TurkeyMan
Copy link
Contributor Author

I'm also gonna say that, unless someone else decides to spend a reasonable amount of time on this, I'm gonna keep writing the code the way I have been ;) .. I have no motivation to write code that I can't use in my own projects.

It's not strictly betterC, it's just not making unnecessary use of Phobos, and not backing into a corner that it needs to be completely rewritten.

If there's a single API you're unhappy with because it supports betterC, can you point it out?

@p0nce
Copy link
Contributor

p0nce commented Jan 20, 2019

I'm just not interested in the result of this WG, because I've rewritten quite a lot of phobos for my own purpose, and it seems unlikely I will be able to use the image library (that was expected). I was expecting something much more minimalistic and D-ish. I think we disagree on how minimal things should be.

@TurkeyMan
Copy link
Contributor Author

So, is this comment accurate? Discord discussion seems otherwise... maybe?

@p0nce
Copy link
Contributor

p0nce commented Jan 20, 2019

Well, no. I change mind very often.

@drug007
Copy link
Contributor

drug007 commented Jan 20, 2019

So you will be interested in the result?

@p0nce
Copy link
Contributor

p0nce commented Jan 20, 2019

Yes.
From the very beginning I thought that "hell no", (because I'm not a fan of collegial anything) but I've written what I need here https://github.com/DlangGraphicsWG/documents/wiki/What-will-make-YOU-actually-use-reject-the-image-library? . If it improves measurably the performance of my products without much integration headache, why not use it?
(edit: so yes I "started" the effort with no intention to use the result, but well I guess everyone can change opinions)

@TurkeyMan TurkeyMan merged commit d2c4f47 into master Feb 16, 2019
@TurkeyMan TurkeyMan deleted the better-c branch February 16, 2019 21:56
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.

3 participants