« first day (3533 days earlier)      last day (1419 days later) » 
00:00 - 14:0014:00 - 00:00

12:28 AM
@TheodoreBrown Would you like me to run the STV calculations for wiki.php.net/rfc/shorter_attribute_syntax ? STV isnt as simple as it seems :-)
@bwoebi You need to vote again, because @TheodoreBrown used a silly format for the names of the votes (not in an algorithmic form)
 
 
1 hour later…
1:54 AM
@Derick Thanks! I read through the wiki article to make sure I understand the algorithm, but I would be happy to have you double check the results when voting ends.
 
 
2 hours later…
4:08 AM
If I really don't like the very very verbose @@ option should I not take a vote on third choice then? Is it better in third choice vote to give a vote or not?
 
 
2 hours later…
5:50 AM
you can't tweet about @@!
 
Space regex ・ *Regular Expressions ・ #79709
 
6:09 AM
@beberlei so you can newer tweet about annotation with two @ :D I've considered it to weird from the beginning
And yet you've chosen it as a second choice? Hmmmm I thought the <<>> would be your 1st or 2nd choice
 
Git mergin'.
 
6:29 AM
This is so sad. Everyone wants attributes but people cannot agree on the syntax. And all just because of the error suppression operator. Attributes will be used much more often than error suppression. This is for PHP8, so let's do something nice for once and break BC. Change the error suppression operator and make attributes use @. The feature deserves it.
 
7:02 AM
and then reimplement error suppression as @silence. problem solved.
 
why would php8 still need the error-suppressor?
 
@cmb Is it too early to remove the optional parameter of curl_version() that you deprecated?
 
@Sjon @mkdir, @unlink, @fopen...
 
Reproducible segfault in error_handler during GC involved an SplFileObject ・ Reproducible crash ・ #79710
 
@beberlei How about try catch? In other hands why would someone who needs error-suppressor need PHP8?
 
7:11 AM
@beberlei I'd want errors for those. Although we did indeed write a custom locking mechanism to implement that correctly
 
@Crell Lol, I had a similar experience with the nullsafe operator xD
 
typo ・ Website problem ・ #79711
 
mkdir could fail because a file is in the way, unlink could fail because of perms and I don't think fopen failures should ever need be suppression
continuing as if they didn't fail would only get you into more trouble
 
@Crell Short-circuiting as in how?
 
cmb
@MátéKocsis deprecate in 7.4, remove in 8.0 – short timeframe, but in that case that may be okay. I think it's worth a PR. :)
 
7:14 AM
Do we consider removing errors in the future? Or conversion of them to exceptions would be reasonable? Today morning I was thinking why we still need set_error_handler and set_exception_handler to complete it there is a register_hutdown_function and pcntl_signal
Is it a hard question or nobody cares?
Regarding the attributes I was thinking in the morning how Rust outline attributes work and if PHP can benefit on that. I know it's not a topic for now, but was wondering if for simple scripts it wouldn't benefit to get rid of all those or part of register some crash handler functions
 
I think not all errors are exceptional. I'm trying to find a good example, sometimes you simply need a side-channel from a method besides the normal return/exception
 
@Sjon What you do with the side-channel? Do you mean logging errors?
 
@TheodoreBrown you didn't vote yourself, are you waiting or do you not have voting karma? You already worked on the number seperator RFC and much more, i thought you can vote?
@Sjon yes fopen, mkdir and such, they essentially need go style multiple returns
its not an exception if fopen fails most of the time, it is expected with probbility %
 
Getting back to what I started with the annotations, my reasoning is deeper I think cause I don't especially like PHP standard library functions which have impact on the engine. I'd love to get rid of them or at least get a way for using the same functionality but without calling register functions, like for eg. a dedicated core attribute on a function which registers error/exception handler
 
same as you wouldn't throw an exception in an HTTP client when the response is >= 400, but you check for status code
 
7:24 AM
We have plenty of functions in standard library which are shorthands which I think in rare cases are not needed and yet the whole standard library is loaded cause I need for eg. error/exception handling.
 
crazy idea: allow multiple return types in php, but only as a side channel to return warnings/notices and so on. $fp, $err = fopen(). the function call knows if its called in "return error as secodn return value" or in "throw error to zend error stack" mode
 
@beberlei that's not crazy, go has something like that
it can be achieved with array descructor via list operator
 
yeah, but they have it explicit, you need to return a, b and then on the call iste a, b = foo()
 
but the benefit would be then if it would be possible to simply return multiple values and assign for instance only first or part of them
 
@brzuchal I think mkdir failing because a directory already exists is a good example of why you'd need something besides a boolean success, and an error
 
7:28 AM
@Sjon if that could be a dedicated exception you could simply ignore it right?
ofc, the code gets longer but more clear to me
currently you need @mkdir to suppress errors, cause cannot catch them
 
@brzuchal currently it'd be an "ErrorException" and I need too much logic
 
@cmb PR is done. I did it as part of the default param value fixes in ext/curl, but I committed the removal separately, at least.
 
@Sjon fine, but we all talk about function which is also easy to wrap in another function in a safe form and use it widespread, probably FW's have some similar implementations - what I mean it's easy to wrap it and forget about the additional logic
That's only one reasonable example fopen + mkdir
And yet, we need the whole erroring functionality for just those two cases
 
a simpler idea, a new function silence() :3v4l.org/v3fOY
 
@beberlei I was thinking over that 5m ago also
 
7:34 AM
though imho the problem is that the error is supposed, you really want to have access to the error message and code
 
but I'd say not a function
it could work similar to assert()
but suppress errors
$error = suppress($fp = fopen('non_existant_file'));
But as a language construct working like assert which runs the ast given and fetches the error
No matter the construct is called
But you know what I mean
 
I don't think that'll work - I want an Exception when mkdir fails because of perms but not when the directory already exists
that's why error-levels are useful and you'd lose that when converting all errors to exceptions
 
This would not convert anything to exceptions?
 
@naderman no, this would not, it just hides the side-channel for errors and allows to easily catch them on aside
sort of try catch for errors respecting error level
it could even be upgraded with second argument for error level with which the construct works, this would allow to avoid the need to manipulate on error level
I've seen this a lot, when error level is bumped to catch errors and then restored to previous
That's ugly
$error = suppress($fp = fopen('non_existant_file'), E_ALL);
Would be similar to put a $fp = @fopen('non_existant_file)
or a variant with multiple values:
$fp, $error = suppress(fopen('non_existant_file'), E_ALL);
But multiple values is a different feature
Just thinking out loud
 
7:50 AM
I'd prefer to remove all errors and replace them with specific exceptions. So I'd get Php_Mkdir_DirectoryAlreadyExistsException and could ignore that
 
try {
    mkdir('/');
} catch (DirectoryAlreaadyExists) {
    // no-op
}
@Sjon like that?
 
yes - but I prefer some sort of namespace in the Exception
 
I don't care much right now that's a different topic
Thats just an example
 
@Crell Haha they even used some of the same examples :D
 
Well it's better then to catch what you need to handle and then add finally block, right? Would that work?
 
7:57 AM
sure, both work - but it's currently not possible with plain PHP
 
8:17 AM
@Derick thanks for the heads up
 
8:32 AM
!!rfcs
:(
 
@Girgias He ded
 
Yes, so sad :(
 
@Gordon I'm pretty sure that I will use error suppression more often than attributes. Except maybe for compile time annotations, which shall not steal my attention too much.
A subtle comment-like syntax is best for that, thus #[] is my choice.
@brzuchal I would be okay if we converted all stdlib func errors to exceptions and made @ an exception (not throwable) silencing operator.
 
@bwoebi That's actually a neat idea, never thought about that
 
At least most of these which are currently error level E_WARNING
 
9:03 AM
@MátéKocsis with OCI8 getting stubs is only pgsql the last extension not having stubs?
 
@Girgias Yes! :)
 
\o/
 
+ a few SAPIs define a few "additional functions", which also doesn't. I have the PR to address this, but generating the "zend_function_entry" array for them is problematic.
 
@TheodoreBrown I have a script. And the Wiki article isn't all encompassing :-)
 
ugh php is so bad
I hate php
 
I had a dream that I was cleaning up run-tests, so I woke up - hit my computer and finished up my PR - https://github.com/php/php-src/pull/5705/files

@NikiC I've addressed your comments and I consider the PR ready for merge.
 
The only way to create a certificate request with subject alt name is ... to generate temporary openssl config file???
 
9:56 AM
@bwoebi good idea :)
 
I have an Car abstract class an some other classes that are extendining this class
Let say in some of classes that are extenindg Car I have an object front_parts
Question: How Can I check in the Car class if any of the classes that are extending has object front_parts defined?
I do know about property_exists($car, 'front_parts') but I don't know how to use it in this case
 
There's a town near me called Irlam. I have to go do a job there later today. So far, autocorrect has caused me to tell colleagues I am doing jobs in Islam and Iran.
2
 
@luffy it sounds like whatever problem you're trying to solve, that object orientated programming is not helping you solve it.
 
You got slapped in a face with a wet fish by autocorrect.
 
Does anyone have php compiled where they can easily run a benchmark against the jit please?
git clone github.com/phpbench/phpbench
cd phpbench
composer install

# Make sure jit is enabled
php bin/phpbench.php run --tag=with_jit --store

# Disable jit
php bin/phpbench.php run --tag=without_jit --store

# generate the difference report
php bin/phpbench.php report --uuid=tag:with_jit --uuid=tag:without_jit --report='{extends: compare, compare: tag}' > difference.txt
 
10:02 AM
Only copy this ?
 
that's the commands that need to be run to run the benchmark suite twice, and then generate the difference report.
@DaveRandom take care of yourself mate. Also, I do not recommend damaging yourself.
 
@Danack Yesterdays master branch compilation is enough?
 
@brzuchal yep, any since the 2nd of June would be fine.
context is phoronix.com/… or a picture:
 
@Danack I do know what I am doing in this case. If get this done, it will save me a lot of coding. My question is more how it is done
 
10:09 AM
I thought I could use objectHasAttribute in the main class and it will trigger for al classes that are extending bu I am not sure
 
which is spocks_eyebrow.gif
 
@Danack Crap it requires Phar
 
@brzuchal oh.....you can probably download the built version from github.com/phpbench/phpbench/releases into the phpbench directory, and then it should run...I guess.
@luffy "If get this done, it will save me a lot of coding." I doubt that sincerely, But also, if you make an example like sscce.org it would be a lot easier for people to help you, rather than to just try and describe how you want the code to be.
@brzuchal oh that's a good sign. the version of phpbench they're using is over 4 years old. I'm sure there haven't been any bugs fixed in it since then.
 
@Danack building it again cause I had it witout a Phar soo it compiles in the background anyway for now
 
cool.
 
10:22 AM
morning
32C today, joy
 
Further run-tests update
I've updated #5707 - https://github.com/php/php-src/pull/5707/files - to reflect @Kalle comment on snake_case, so my functions (at the bottom of the file) now use snake case.

I've rebased this #5707 on top of the latest commit for my first PR #5705 - https://github.com/php/php-src/pull/5705

So once we merge #5705, then the diff for the second PR #5707 will look very small.
In essence I've added a few functions at the bottom of the file.

I have a few more hours to spare for php-src, so I'd like to get #5705 merged and hopefully #5707, to keep them small, so I can
 
root@062882cf91de:/src/bench/phpbench# php bin/phpbench.php run --tag=with_jit --store

In Payload.php line 130:

  Fatal error: Uncaught RuntimeException: Class has no method "providePoints" and it is not a callable in /tmp/PhpBenchq59OOq
  :48
  Stack trace:
  #0 {main}
    thrown in /tmp/PhpBenchq59OOq on line 48


run [--filter FILTER] [--group GROUP] [--parameters PARAMETERS] [--assert ASSERT] [--revs REVS] [-l|--progress PROGRESS] [-b|--bootstrap BOOTSTRAP] [--executor EXECUTOR] [--stop-on-error] [--php-binary PHP-BINARY] [--php-config PHP-CONFIG] [--php-wrapper PHP-WRAPPER] [--php-di
@Danack ^^
 
@brzuchal hmm. is that from a downloaded phar version or from git? the git version doesn't seem to try to call providePoints in Payload.php line 130
 
from a git
 
@brzuchal that's super weird then. As that line shouldn't throw that error, unless I'm completely miss-reading it.
oh I am.
....the exception is happening elsewhere...
Hal, changing a light bulb.
 
10:40 AM
Exception trace:
  at /src/bench/phpbench/lib/Benchmark/Remote/Payload.php:130
 PhpBench\Benchmark\Remote\Payload->launch() at /src/bench/phpbench/lib/Benchmark/Remote/Reflector.php:96
 PhpBench\Benchmark\Remote\Reflector->getParameterSets() at /src/bench/phpbench/lib/Benchmark/Metadata/MetadataFactory.php:74
 PhpBench\Benchmark\Metadata\MetadataFactory->getMetadataForFile() at /src/bench/phpbench/lib/Benchmark/BenchmarkFinder.php:73
 PhpBench\Benchmark\BenchmarkFinder->findBenchmarks() at /src/bench/phpbench/lib/Benchmark/Runner.php:116
 
@NikiC Yeah I think so, always annoys me too ...
 
@brzuchal I'll do a PR to the project. The actual info is in the previous exception, which has been lost...
 
@Danack +1
 
oh strewth.....there isn't a previous exception, the info isn't there....it's in another process...
 
yes, there is a template for benchmark class
 
10:47 AM
@brzuchal actually, can you try just turning the jit off, and seeing if the code works then. There is a non-zero chance it's a bug in PHP.
obviously being able to tell where the bug is is an issue all by itself, that phpbench should fix but worth checking that first.
 
root@062882cf91de:/src/bench/phpbench# php -i | grep jit
auto_globals_jit => On => On
pcre.jit => 1 => 1
only got these
 
cool. thanks.
 
yw
Wow what a race, shorter attr syntax #[] === @@ for now
 
11:11 AM
@brzuchal but #[] wins by the seconary vote
 
@bwoebi Let's see, personally I'm a huge fan of #[]
Funny thing there is 25 votes, 24 votes and 20 votes in first second and third choice respectively.
 
too complex for me
 
11:31 AM
@brzuchal sorry! I just made it 11 vs 10 :)
 
@beberlei No, I don't have voting karma.
 
cmb
that's the trick: invent a voting system so complex that nobody is able to verify the results you are presenting. :P
 
@TheodoreBrown you should have though
 
STV isn't that bad from a voters perspective: rank in order of preference — It's harder for counters, but it's pretty much the fairest system for this kind of stuff, with very few seats.
 
@beberlei Should I ask for it on list?
 
11:38 AM
I don’t remember what the process is
 
@TheodoreBrown I think @salathe has the power....right now is not a good time to discuss voting rights imo
due to other rfcs.
and the process is vague and open to severe problems, which we really ought to address before they manifest....but that also will be an 'exciting' discussion.
 
@Danack what do you mean discuss? Thedore only wants to request voting rights for himself
 
Jun 4 at 8:24, by salathe
@IluTov Sure, if you want a short email thread of folks arguing between themselves on who gets to vote, followed quickly by total silence and no action on your request. :P
 
@beberlei yep. And then in about ten days when the voting starts on another RFC, internals could be brigaded by a whole load of pmj's friends asking for voting rights.
 
Jun 4 at 8:25, by salathe
I'd suggest getting a php.net VCS account instead, if you're planning on being a longer term contributor to the project. A side-effect/benefit of that is you get to vote.
 
11:42 AM
@Danack theodore already has worked on one rfc for 7.4 that was accepted, i believe that counts as a higher bar?
 
I absolutely think he should be able to vote.
aka what salathe said.
 
@beberlei I worked on two successful RFCs for 7.4 (numeric literal separator and curly brace array access deprecation)
 
@TheodoreBrown my bad i only remembered the numeric literals
 
I worked on RFC accepted for 7.2 but then didn't get the vote karma automatically.
 
just ask for a docs "PHP CVS" account Theodore
IMO, people that have had a succesful RFC should also get a vote
or rather, get a @php.net address (which gives voting rights)
 
11:51 AM
@Derick I'm not sure how. Are there instructions about where to ask?
 
let me see
there is a page
 
Wes
mornings
 
\o
 
unable to go on multiple lines with the new lambda syntax ・ *General Issues ・ #79712
 
@TheodoreBrown There are trick question in that form, please read everything first :-)
 
11:56 AM
It may make you doubt your sanity
 
Yeah like that form :D
 
@Derick :D Should I send to the PHP Group or the Doc Group? Seems like it should be the former since my contributions have all been to PHP core, not documentation.
 
Can somebody change my username on php.net? It was ilijatovilo but I'd like to change it to ilutov (as it's easier for folks to remember). I can't seem to change it myself.
 
@IluTov Sorry, can't be done
it's a primary index in so many things; changing it by hand will cost hours, and it will go wrong
 
@Derick Bummer, thanks anyway :)
 
12:05 PM
I can't find your php.net account though?
 
@TheodoreBrown that doesn't really matter, since it will be one of us approving your account
@Derick Doesn't look like there is one yet. There is a wiki user ilijatovilo though.
 
@salathe Oh sorry I thought the two had to be the same
@Derick What's "Requested Password" in that form?
 
your password
I don't know what it is.
 
@salathe Okay. I submitted the request to the PHP Group now.
 
@TheodoreBrown Approved
 
12:13 PM
@Derick \o/
 
Now you get to merge your own fixes!
Thu, 18 Jun 2020 11:22:28 +0000 <- that was the request time
 
experienced tea drinkers, do I put milk in earl gray tea?
 
that's... nearly an hour off
the date on that server is so.... wrong
 
@Derick Lol, the "requested" threw me off xD
Thanks
 
@Derick Thanks!
 
12:15 PM
@Tiffany I don't if it's good flavourful tea. Try it without first, and if you like it - don't add milk :-)
 
@Derick it's twinings, but it's import
though, it might be produced here, I dunno how that works
 
I... drink loose leaf tea
I'd try it without, and, if you like it that way... don't add milk :-)
 
@Tiffany Of course you do. Bit of good honey is nice too
 
general rule: flavoured tea → no milk
standard black builder's tea → milk
 
I have honey from Sardinia, extended family makes it themselves. Absolute heaven
@Derick How dare you - actually you're correct xD
 
12:17 PM
why would you want to destroy the flavour of the flavoured tea? that is herecy :-þ
 
Never mind I forgot how to do strikethroughs
 
@Jimbo ---
 
Okay, but earl grey doesn't count as a "flavoured" tea though
 
builder's tea without mlik is undrinkable
 
Try with milk, honey, lemon, or any combination of them or none of them. Find what you like most, and have it that way forevermore. :)
 
12:18 PM
@Jimbo The twinings stuff, perhaps not. My loose leaf tea, definitely
 
I learned the hard way to always drink English breakfast with milk - I had a stomachache for a few hours, hadn't eaten anything either
 
don't add milk and lemon at the same time though
it makes the milk curdle
 
@Derick sshhh
 
But. It. Is. Chemistry!
 
I can accept driving on left side but milk in tea - never. :P
 
12:19 PM
/off-topic - Other than the ini settings, is there a way to tell if jit is doing anything or not? Or has functions like php.net/manual/en/ref.opcache.php ?
 
run it with valgrind, and if valgrind complains, the JIT is running ;-)
 
On the off-chance anyone has enabled TLS for RabbitMQ and knows what this might be... would be great. Golang though.
1
Q: tls: handshake failure when enabling tls for RabbitMQ with streadway/amqp

JimboI'm attempting to connect to RabbitMQ with amqps:// in Go using streadway/amqp. I can connect successfully with amqp://. When enabling TLS and using amqps:// I get the following error: panic: remote error: tls: handshake failure RabbitMQ is running in docker with the following environment vari...

There's no Golang chat here
 
@Derick Any chance you can share your STV script? Otherwise I was thinking of writing my own.
 
I can, but I've only implemented what I needed.
and for it to work, you need SSH access to the wiki machine...
 
@IluTov your email went to my GMail spam box
(about PHP.net account)
 
12:32 PM
@Girgias Ah damn it... I didn't think it would matter here.
No clue why @me.com mails go to spam
 
Who knows
 
Can I change the e-mail address in that request to tovilo.ilija@gmail.com?
 
@Danack that was something i requested 2 years ago to see at least in opcache_get_stats() but nothing there yet afaik
 
My wiki account has ilija.tovilo@me.com but I can't change it there either xD
 
Well, this is going well. I did a fresh checkout of php master, configured with
...
 
12:35 PM
@IluTov you can change your email address in settings when it's approved
 
cmb
@IluTov you do no longer need that account, once your php.net account has been approved :)
 
how do I configure and enable jit in php? From fresh checkout of master.
 
cmb
@Danack opcache.jit_buffer_size=16M (doesn't work for debug builds)
you can check than php -i | grep JIT
 
@Tiffany @cmb Ah I see, thanks :)
@salathe Is it possible to change the email of my php.net user that I just requested to tovilo.ilija@gmail.com?
 
12:47 PM
@cmb thanks. The problem was I didn't have the opcache module loaded, so that was saying "PCRE JIT Support => enabled" but the ini settings weren't read.
 
Remember "the past"? Here's a blast from it youtube.com/watch?v=KXK1ZOvc6SY
 
(just wait until your account is approved and you're able to access the settings, then change it then)
 
@Tiffany Ok cool, thank you :)
 
@Derick if something wins in STV on a first poll with a difference of one then it wins, right?
 
no, it needs to make quorum
 
12:54 PM
Oh, great
I love it
 
It feels like a Friday to me.
 
For me it's Friday from Monday :D
 
@Jimbo that's a very thorough description. Did you try tcpdumping the connection, maybe wireshark can give you a hint
 
=)
@MarkR I need some new music. Have something I've possibly never heard? =)
 
@StatikStasis did you listen to the one I linked you yesterday?
 
12:57 PM
Oh no- let me go look.
 
that album has become my programming music...
 
Sounds like a busted speaker =P
 
lol
 
How are the cats?
 
@StatikStasis youtube.com/watch?v=Upqzi8I04jA this is a bit different, the other songs on the album are pretty good, but World Eater and Dominatron are probably the better ones
@StatikStasis trying to get one to lose weight, so I've been restricting their eating, and they woke me up at 4:30... need to go back to the weight loss food
(in other words, they're cats)
my niece is fostering some kittens though, going to go see them today :D
 
cmb
1:11 PM
 
@cmb Yay, thank you!
 
cmb
yw
 
@Tiffany awesome!
 
it almost took to long for the reddit thread having a comment "why not just @ like everyone else?" :p
 
Let me pull up the meme from last time
 
1:45 PM
Should I avoid using goto? I've never used it in PHP but I feel like it could be useful. I've always heard that it's a code smell though
 
I've just posted on internals, but I'm not a fan of @@ purely from the long term view that if we ever have "optional" types it would almost certainly be @ at which point it's @@@ and that just looks stupid
 
@MarkR thats thinking a few steps ahead though :) you could turn it around and do @@! to force validation for example :p
(as in, there is always a solution)
 
I can't think of a situation of us adding validation where we wouldn't want it by default... because that would be the standard with optional types such as with generics. What's the latest behaviour if the class is missing when you try to get the attributes?
 
@Alesana use it where's it useful. It's fine in PHP, it's only really bad in languages that don't have automatic garbage collection.
 
@Alesana I don't remember the details, but there have been a few times where I thought converting goto into an existing structured form was worse, imo.
 
Wes
1:52 PM
my very horrible parser stuff uses goto just because that way was much more readable
 
What do you guys think is cleaner
 
for single controller scripts i sometimes for fun do <?php at the top, then html below that, and use "goto controller; goto view;" to jump around. but thats mostly to be silly
 
... but why?
 
1:53 PM
Just use return false;
 
My IDE just told me not to have too many returns lol
 
For simple returns like this, I would just return false instead of goto. If the return expression was complicated I would consider goto.
 
wtf
 
Ah okay. That makes sense to me
 
Your IDE is dumb then
 
1:54 PM
It was sonarlint
 
@Alesana Your IDE needs a slap then
 
@Alesana Early returns are fine :)
 
Also, you can combine some of the ifs, probably.
 
Although why would you even have a return type if it only ever returns false?
 
@LeviMorrison I originally had that but it was a lot less readable
 
Wes
1:54 PM
yeah no just return false :P
 
@MarkR It's a rule that implements an interface, there's a lot of rules
It's the only one that doesn't return true because it is special
 
if ($managers->isEmpty() || ($userLoginHistory = $this->getLoginHistory($managers)) === null) {
    return false;
}
Whether this is an improvement or not depends on your worldview :)
 
@MarkR What do you mean by optional types? Are you saying you'd want to use the suppression operator to hide an autoloader failure if an attribute class doesn't exist?
 
Wes
maybe read about the "return early" pattern also
 
@TheodoreBrown Yes
 
1:56 PM
@LeviMorrison I originally had something like that but it made it harder to understand what was going on
 
yay, all openssl tests are passing
 
@Wes I think I use that quite a lot to avoid else or just to make the code more readable. I didn't know it was a pattern though
 
Wes
tl;dr avoid nesting ifs and avoid else{}
 
@LeviMorrison I do not think it's particularly readable to hide assignments in a secondary condition in an if - assigns in ifs are fine, but please at the very start of it.
 
00:00 - 14:0014:00 - 00:00

« first day (3533 days earlier)      last day (1419 days later) »