« first day (5018 days earlier)      last day (151 days later) » 

10:28
@geertjanknapen it's SO, not the nicest community...
11:20
Yeah, that's true, but that does give off the impression that someone's opinion is more important than the actual quality of the question, which contradicts what SO stands for.
I would like a required reason field when downvoting, just downvoting without an explanation as to why does not make for better questions..
 
4 hours later…
15:06
I am using symfony and wanted to call one controller action (which is private) into another controller. It sounds terrible, I know but that private function code I don't want to duplicate. that's only motto I've. can we use some helpers OR other ways through which it can be possible?
What does the code do in the private method?
some API calls and imp. checks
15:26
It should probably be a service or services instead and passed into both controllers
16:14
Do you have any good example for reference?
well! even if I create a service, how can I access private member of one controller into that service?
You could use a trait.
oh, I can put that private fn inside trait and use that trait wherever I need?
16:32
@Exception My guess is that your controller is doing things / is responsible for things it shouldn't :)
But who knows without knowing the code
What I do know however that traits are almost never the solution
 
4 hours later…
20:50
@TimWolla I actually cannot find any reference to this in the RFC howto
What is prohibited from further changes? The RFC or the implementation, or both?
@QuolonelQuestions The specification. The implementation is not voted on, but obviously it is expected to match the specification before it is merged.
The implementation does not need to be finished before starting the vote.
Well that's entirely contradictory to the howto

> make it clear whether or not the current implementation is being voted on
@QuolonelQuestions Your excerpt is missing context: "Consider whether the feature concept should be voted on before you invest time in implementation. Or perhaps an implementation is needed to show the feature is feasible; in this case, make it clear whether or not the current implementation is being voted on." This is referencing complicated languages features that require proof that they are actually implementable (e.g. generics).
Ultimately, it is up to you to start the vote. If the vote is accepted, and then turns out not to be implementable according to the specification, then it just won't get merged.
21:25
I don't think the context really changes anything. It's still suggesting you can vote on the implementation. I also recall seeing doodles with titles like, Merge X as implemented, which seems to be following this advice to make it clear the current implementation is being voted on. I also planned to do this, since I believe the implementation is good and complete, so it can be studied to determine its suitability
21:40
@QuolonelQuestions We generally only vote on technical changes if they are large and/or risky, making them relevant to the community at large. Remember that many if not most of the voters are not C developers.
I'm not a C developer :^)
So, yes, you can vote on implementation, but that was not your question. I've never seen an RFC that specifies to merge code as-is, disallowing any changes. It is not meaningful, because the code is free to change after merging anyway, as long as it has no functional impact.
OK, so getting to the point, my RFC cannot be changed once voting begins (but the code can), right?
Yes
It is generally accepted at typos can be changed (as long as it isn't subject to misinterpretation). If you find a behavioral mistake, you can consult the list to see if fixing it is acceptable, or whether it requires a re-vote.
21:58
OK thanks :)
The howto also says, Update your RFC to document all the issues and discussions. Cover both the positive and negative arguments., but looking at other RFCs published lately (within the past few years), it seems nobody is actually doing this
Nevertheless, I am considering doing it, for posterity
@IluTov I invited you to comment on why you would reject my proposal. Did you just not get around to it or do you not have anything you want to share?
@QuolonelQuestions How so? I specifically push for objective reasoning when working on RFCs.
So far I think this is the only RFC that has a discussion section. I'm sure there are more, but they are hard to find
@QuolonelQuestions I don't think this requires having a discussion section, but all legitimate arguments being documented somewhere in your RFC. I.e. you can't just ignore issues raised on the list.
Do you have any legitimate arguments for me to add? :)
If not, I'll just add that some people prefer namespaced functions and some think that this somehow promotes the use of static (it doesn't, but they're entitled to that view), which they view as an anti-pattern. That's about all I've been able to gather from the list
@QuolonelQuestions I didn't say that I would vote against it, just that I am not particularly in favor. I don't see a big value-add. I understand that static methods have some benefits over functions, namely that they can have visibility and work properly with autoloading. However, I find inheritance and polymorphism for static classes questionable, and hence it is almost equivalent to a final class, where you just don't use non-static methods (except for new).
I generally don't vote against RFCs if they don't pose any problems (now or in the future), and if the implementation is simple enough.
22:16
Ah, sorry. I misinterpreted your position as against rather than abstain
I agree, it's not a big value-add. It's a small feature. Small, but not insignificant, and about the best I can do at the start of my journey into internals. Hopefully I can achieve something greater on the path forward

« first day (5018 days earlier)      last day (151 days later) »