« first day (1439 days earlier)      last day (3506 days later) » 

7:00 PM
They do if they want access to the thread resources....just most of the time inline functions are just doing simple operations?
 
@Leigh huh?
You don't need to define TSRM* if you don't use the tsrm, or if no function you call uses it
lxr.php.net/xref/PHP_TRUNK/ext/standard/password.c#69 <-- not inline (though maybe should be), and doesn't call any functions, so no need for it
 
So, I'm looking at
http://lxr.php.net/xref/PHP_5_6/ext/gmp/gmp.c#379
vs
https://github.com/php/php-src/pull/839/files#diff-6879cdf65c6d3932ab79c03530dbb158R1798

Niki says it needs TSRMLS_D, which is fine, but I want to understand why (GMPG?)
I also don't get why we static all the things
 
0
A: What is the difference between `while (true)` vs `while (\true)`?

NikiCIn PHP true, false and null are constants, which are protected from being overwritten in namespaces. As such the following is invalid code: namespace Foo; const true = false; // Cannot redeclare constant 'true' It is still possible to overwrite such a constant using ''define()'': namespace Fo...

@Leigh Yes, GMPG
GMPG accesses thread-local variables, which is why it needs the tsrm_id
 
@Leigh Just to be clear what do you mean by "how come inline functions don't need to define TSRM*" particularly - what are you meaning by inline?
 
ok, obviously i understand why 2 threads accessing something global can cause a race condition, I think I was more confused by other functions not having it
because they "might not be inlined"
like, the function I linked, copying a zval, feels to me like that should be thread safe...
unless that's buried down the macro-chain somewhere
 
7:10 PM
@Leigh which function?
 
github.com/php/php-src/pull/839/… isn't an inline function...but anyway even if it was it's not the inline-ness that makes the difference, it's the fact that it's using thread specific resources.
 
@Leigh convert_to_long will do a TSRMLS_FETCH() in the cases where it needs the tsrm_id
 
@Danack you probably missed the thing yesterday where we discussed that even if a function is declared inline, it might not get inlined, so you shouldn't assume it will
anyway fine, thread stuff is handled deeper down, makes sense then
 
7:16 PM
@ircmaxell btw, I just realized that since php 5.6 the constant substitution isn't just inefficient, it is actively wrong. See 3v4l.org/viHTv
 
@Leigh The macros to access the thread resources would still need to have access to the stuff that is defined by TSRMLS_D whether or not it's inlined....right?
 
@NikiC I didn't do that
 
> W3webschool on of the best online and offline training institute who want to become expert in web design,Php/mysql ,SEO,wordpres and also HTML/css3 with 100% practical classes. Source - reddit
 
@NikiC wait, why is that wrong?
 
@ircmaxell use const AltMath\M_PI
it imports the M_PI constant from another namespace, where it's defined as 3
 
7:18 PM
...?
yes, and that's wrong... how?
 
but in the code it uses the global M_PI with value 3.14
instead of the imported one
oh wait
I got confused there with the different outputs
 
yeah, 5.6 is doing the "right" thing there
 
yeah sorry, I just looked at the first result, turns out that hhvm does it wrong
 
yeah, I'm bringing that up in IRC now
 
@define(__NAMESPACE__.'\true', false); -- how to make your coworkers hate you?
5
 
7:22 PM
so php is just inefficient after all, and not wrong :)
 
So I am joining and fetching tables.
Problem is, it's a question answer type thing, so one question may have multiple answers.
 
@NikiC what's inefficient? I haven't looked deeply at it
 
As a result, a simple oci_fetch returns the same question twice because one of the questions has two answers.
how do I fix this?
 
@ircmaxell we just substitute less constant at compile time than we could
 
blame opcache for that
but it's still more efficient than the alternative (looking up constants on every access)
 
7:24 PM
@DemCodeLines We've talked about your SQL before on this project. If you join the answer table to the question table, then you're going to get one row for each answer. You have to either a) not join the answer table, or b) use GROUP BY to group the rows on the questionId.
 
the constexpr substitution is only incorrect if you use use const Foo/BAR as true. Not sure if we consider that valid code ^^
 
@Chris We have talked multiple times, but every time, everything seems to be perfect while you're there and as soon as you leave, something breaks and I come back here asking questions lol
Anyways, group by on the questions table?
 
actually...
thinking about it, what if we added a "post-compile" task list. That way when you parse and compile a file, it'd also return a list of things to perform afterwards. Without opcache, that list is directly executed. And with it, it's executed after symbol tables are restored. That list would be used for inheritance checking (instead of late binding as now), but also things like constant evaulation and lookup, etc...
@NikiC i don't know what HHVM is doing with use const, but it's fubar: 3v4l.org/udEv1
 
@NikiC about special casing and using _ui functions, did you mean use _si? (-1 as long will be > (GMP)0)
 
@Leigh then special case for IS_LONG and LVAL >= 0
as there's no add_si or sub_si
 
7:37 PM
@DemCodeLines Yes, or query for only questions and skip the join, then loop the questions and get answers for each as a separate step. It really depends on what you need for the page at the moment.
IIRC, there was a crazy query where I (semi-jokingly) wrapped them all in MAX -- the reason that was done is because you were doing a GROUP BY on the questions table... to prevent getting multiple rows. My non-serious suggestion got posted as a question, to which you received a good answer. I don't understand how you're still dealing with that :p
 
seems like Andrea and Stas want to keep asp tags ...
 
I've personally never seen ASP tags in use on any code I've ever touched in 15+ years of development, nor in samples, tutorials, complete third-party libraries, or anywhere else. If someone is actually using that, they've done a remarkable job hiding it from the world.
 
bet it's being used on windows somewhere ...
 
The only place ASP tags exist in my mind is as a not-very-interesting footnote in PHP documentation.
 
deep in some intranet or whatever ... it doesn't matter, we should ditch it anyway ...
 
7:46 PM
I think narrowing down on ambiguity (like removing alternate tags) helps create better developers - imagine a junior WTFing the first time he sees that shit.
lets get rid of semi-colons as OK syntax for case/default!
 
@Leigh yeah, I don't get why that's even a thing
 
I wrote my first lines of PHP 13 years ago, I never knew it existed until you mentioned it
 
Maybe it's there so people can write <? case 'foo' ?> ?
not that it makes a lot of sense, given how the templating syntax for if etc all require a trailing colon
 
no no, it's <? case; 'foo' ?>
 
I didn't know it existed until I was trying to break your rfc
 
7:49 PM
I think you meant <% case; 'foo' </script>
4
 
btw. if they weren't definitely used, I'd kill <? ?> too… (but not <?= ?>)
 
@PaulCrovella STOP DOING THAT!)
 
)
 
7:59 PM
now we have a closed parenthese without an opening one
@NikiC fix it! you're the only one who can save us!
 
1 message moved to recycle bin
you're welcome
 
:-(
 
Why can't we just make true/false/null un-overwriteable? @NikiC
 
It basically is, being a constant. Seems to me that the ability to "overwrite" true/false constants in a namespace is a byproduct of having them be constants rather than keywords. You'd either have to modify the entire way they're used, or build a mechanism into runtime constant defining that defends important constants.
Which would probably be smelly code.
Yah, short open tags are the worst. <? $someVar ?> will drive you nuts trying to debug (missing the = for echo), those are my least favorite problems, when you typo-omit some punctuation mark and it breaks your code in a non-obvious way.
 
8:15 PM
@Chris I wouldn't object against them being keywords…
 
@bwoebi they are unoverwriteable. just not consistently so ^^
 
@NikiC yes, I'm talking about the namespace-trickery…
 
Into the realm of speculation... surely they started out life as constants for some historical reason, and there's been no incentive to change it, right?
 
@bwoebi which one?
 
57 mins ago, by Chris
@define(__NAMESPACE__.'\true', false); -- how to make your coworkers hate you?
 
8:19 PM
heh, add echo '<script>window.undefined = "your mom!";</script>'; and you've pretty much shut the whole thing down
 
user1994804
Howdy Gang, I've got one that's as simple as they come but I can't get it to work...
 
fuck this bullshit
 
I am going home
 
Still at work?
 
8:21 PM
@tereško great idea :-)
@tereško what's happening at work?
 
user1994804
$Query = "SELECT * FROM `table1` WHERE `ID` = 1";

if($result = $DB->query($Query) == false) {
echo 'LINE 158 DID NOT SELECT CATEGORIES';
} else {
echo '161 SUCCESS';
var_dump($FetchCaResult);
}
 
user1994804
I Get '161 Success'
 
@bwoebi slavery ..
=P
 
user1994804
output but var_dump gives me bool(false)
 
user1994804
WTH?
 
8:22 PM
precedence.
 
user1994804
?
 
ThW
@YourAdrenalineFix the == is executed before the =
 
$result = $DB->query($Query) == false vs. ($result = $DB->query($Query)) == false
 
user1994804
Ive even named variables crazy names (not as shown above) to ensure no conflicts. Still no dice
 
user1994804
Sorry @ThW I don't "See It". Could you elaborate?
 
ThW
8:25 PM
see bwoebis example
 
@YourAdrenalineFix You're assigning the result of the comparison to $result, not the result of the query.
 
$result = $db->query($query);
if($result === false) ..
 
user1994804
am I missing a pair of () in my if ?
 
ThW
If you have 1 + 2 * 3 - how do you make sure the addition is executed first?
 
user1994804
I don't know
 
user1994804
8:27 PM
I'm newb and trying to clean up and get working some existing code
 
@ThW Der Zaunpfahl war zu winzig… [Just writing German because I cannot express that in English adequately…]
 
@YourAdrenalineFix He didn't ask you a programming question.
He asked you a 2nd grade math question.
1 min ago, by ThW
If you have 1 + 2 * 3 - how do you make sure the addition is executed first?
 
user1994804
Ahh
 
user1994804
I get it... a riddle
 
ThW
<- orders another wine
 
8:29 PM
@ThW use ()
 
user1994804
you place the 1+2 inside ()
 
@ThW I always thought it was popcorn, not wine…
 
user1994804
so Im missing some () in my if
 
user1994804
??
 
@YourAdrenalineFix yes
 
8:30 PM
Hey, I'm running into weird problems
 
user1994804
I tried fixing that but it seemed I created an infinite loop
 
My PHP code has unexpected end of file on my server, but nowhere else
 
user1994804
Hold on...
 
You want two steps. First, $result = $DB->query($Query), then $result === false. The way you've written it, the === comparison is happening before the = assignment. Either split it, so you assign on one line and compare on the next, or wrap the assignment in parenthesis to tell the parser that it should be factored first before the comparison.
 
someone was saying YODA conditionals were useless? Well, they'd have prevented this bug: chat.stackoverflow.com/transcript/message/19082463#19082463
 
8:31 PM
@ircmaxell that's my primary reason for using them
 
@ircmaxell hey… that's the only case where I use them. usually they are useless.
 
to avoid exactly that issue
 
Ugh. I'd rather think about every line of code than use YODA. There's something about it that is so aesthetically displeasing.
 
ThW
:-)
 
Ugh. I'd rather think about every line of code than use YODA. There's something about it that is so aesthetically displeasing.
 
8:33 PM
:-P
 
user1994804
@Chris if you could provide me a proper if statement based off of: if($result = $DB->query($Query) == false) { I would be most appreciative.
 
@NikiC I don't use the religiously, but I try to use them when writing complex conditionals like that
 
user1994804
Otherwise it's gonna be a guessing match
 
is there a ZVAL_BOOL_P() macro?
well, there isn't, but what should I use?
 
@FlorianMargaine Z_BVAL_P()
 
user1994804
8:33 PM
I already tried that and found myself in an infinite loop (Some how)
 
@ircmaxell I don't use them religiously either. and usually only for true, false, null and not other values
 
this is annoying, I see nothing wrong with the code
 
@FlorianMargaine And if you don't know that it's a boolean, use zend_is_true
 
8:34 PM
It's just a class
 
@NikiC if it's an integer, using Z_BVAL_P() is allowed too.
 
actually it's never allowed because it was removed ;)
 
:-P
 
so, I think it's time to revive wiki.php.net/rfc/engine_exceptions
last time around people were asking for more specific exceptions
 
@NikiC ++
 
8:37 PM
@NikiC great :-)
 
Hey guys, this works out fine in any syntax checker, except for on my server
 
@NikiC in phpng you mean?
 
but not sure if / which more specific exceptions should be used
 
Actually running it through php -l or php -f produces this:
[root@server]# php -l /path/class.sudomemoPage.php
PHP Parse error: syntax error, unexpected end of file in /var/www/lib/classes/class.sudomemoPage.php on line 59
Errors parsing /path/class.sudomemoPage.php
[root@server ~]# php -f /path/class.sudomemoPage.php
PHP Parse error: syntax error, unexpected end of file in /path/class.sudomemoPage.php on line 59
 
@bwoebi yes
 
8:38 PM
@AustinBurk no it doesn't
@AustinBurk and it's because of short tags: <?
 
facepalm idk how those got there
 
@NikiC I think it should always use a specific exception name for each "type" of error
 
@NikiC totally agree. I'd not use specific names, because I'd just not want to remember a name for each genre of error…
 
I never use short tags (but clearly I did)
Thanks
 
@ircmaxell I'd not like to have quite such an explosion in exception classes as that tho
 
8:40 PM
@ircmaxell hmm, I especially like the WTFException ;-P
 
It probably makes sense to have more than just one exception type, but it should be closer to three than thirty
 
perhaps not that big, but you'd need more than one or two
@bwoebi it was written explicitly for this line: lxr.php.net/xref/PHP_TRUNK/ext/com_dotnet/com_typeinfo.c#283
 
@NikiC agree, but I assume there always will be some errors which don't fit in either class of errors and then the count of exceptions increments slowly…
@ircmaxell funny
 
@NikiC are you making parse errors non-fatal?
 
@ircmaxell I will do that as well
because now we can
 
8:43 PM
then, you'd need a ParseException :-D
 
the stack unwinding for it is already in place
@ircmaxell Yes, ParseException or SyntaxException (which is better?)
 
@NikiC ParseException
 
actually, is this worth namespacing in the PHP namespace for? catch (PHP\ParseException $e) {???
 
no.
 
Hm, what language is PHP written in? Not PHP :P
 
8:44 PM
@NikiC It doesn't matter much IMHO. Either could be valid
@NikiC yes
 
@NikiC what will you do with the E_CORE_ERRORs? I'd add them to ParseException too (if it's something related to parsing like duplicated function names)
 
@ircmaxell I think the latter might be better in case we want to turn other compile-time errors into exceptions at a later point in time - those are not directly related to parsing, but are often still syntax errors
 
HEY I wonder if one could actually make a PHP interpreter with PHP and then compile that, and then use that to run PHP?
 
@NikiC I'm cool with that
 
@ircmaxell You mean "yes, most people stick their own exceptions into namespaces"
 
8:46 PM
@ircmaxell uh, it even shouldn't break most code because autoloading...
 
@bwoebi are you sure that core errors are what you think they are?
 
@bwoebi no, but it'll cause it to behave weirdly
@NikiC except those declared in global, like the first two in that search ;-)
 
@ircmaxell well, can't make everyone happy ;)
 
@NikiC woops, I meant E_COMPILE_ERROR
 
evenings
 
8:47 PM
I won't touch compile errors
 
@NikiC well, I think namespacing them in PHP here could make sense, as it's a PHP thrown exception...
 
@NikiC but they're parsing errors too?
 
@bwoebi they aren't parsing errors
 
@bwoebi it may be a case by case basis, where each error is reviewed and seen if it can be recovered from
 
@NikiC like (E_COMPILE_ERROR, "jump into a finally block is disallowed") … when parse errors are exceptions, why should these not be exceptions too?
 
8:49 PM
@bwoebi Because it requires too much work
it would require making the compiler exception safe, which it currently isn't
 
@NikiC but what if @bwoebi is willing to do the work to clean up that specific error
and not turn compile_errors into exceptions, but turn that error itself into an exception
 
@ircmaxell It's an all-or-nothing thing
You have to turn everything into exceptions at once
 
if you do it by altering zend_error(), yes
I'd suggest not doing that though, but by changing them into throws at the error location. That way a specific class can be used, and the appropriate message can be set...
 
or rather: once you turned one compile error into an exception and propagated all the necessary exception handling, then turning the rest into exceptions is trivial
 
actually some of these E_COMPILE_ERRORs can also be moved into the AST step. then it shouldn't be an issue?
 
8:52 PM
@NikiC Ah, I get what you mean
 
set_error_handler_and_yes_I_mean_all_the_errors_even_parse_errors_just_tell_me_‌​would_you(myErrorHandler);
 
@Leigh Z_LVAL_P ;)
 
@DaveRandom what do you think? github.com/Ralt/php-src/commit/…
 
@NikiC also, looking through that core errorexceptions file, most of those exceptions wouldn't be needed
because either the error isn't thrown in PHP itself, or it shouldn't be an exception (like E_DEPRECATED)
 
@NikiC grumble, good thing push -f exists ;)
 
9:00 PM
@Leigh shouldn't this commit be rebased? github.com/lt/php-src/commit/…
 
@FlorianMargaine moment, reading
 
@FlorianMargaine why do you think so? I'm generally a "commit little and often" person, and usually I leave all my mistakes and small adjustments visible
 
@Leigh when in your repo, sure, but when pushing to the php-src main repository, your mistakes and small adjustments will be part of php-src history... which isn't pretty imho
I've already had derick ask me to rebase small changes like that, and I think he was right
 
9:04 PM
People usually condense into a single commit when they merge anyway
 
not sure what's the general policy
hm nope, they just click the "merge pull request" button :P
 
don't rebase
when you do the final merge, squash it
 
At this stage, I'd prefer people who are reviewing the PR can see that mistakes were made, there was feedback, mistakes were fixed
tbh, I should be paying Nikita for all of the time he spends helping me with the little things
 
ok, so you'll squash before the "merge pr" button is clicked?
 
don't click the merge pr button, but instead do it manually (you should be doing that anyway)
when you issue the final merge command when doing it in cli, just add --squash to the merge command
 
9:06 PM
Ok
 
that'll merge it as a single commit
 
I specifically requested no repo access, I get away without peer review at work, but I wouldn't want that here.
 
Yeah I'm the same :p I still have it but won't use it...
 
9:07 PM
@Leigh get repo access. You can still do peer review via a PR before merging it yourself
also note that --squash will lose contributor info, so only do that on your own stuff
 
@FlorianMargaine Seems like you should be able to streamline read_dimension and has_dimension a bit - maybe into something that returns a xmlNodePtr, for has_dimension you can just check for it returning NULL, and for read_dimension you can create the return object?
 
@ircmaxell I'm actually a little nervous about having it to be honest :)
 
fair enough
 
@ircmaxell omfg, why did no one tell me about --squash?
 
@DaveRandom hm indeed
 
9:09 PM
@NikiC :-D
 
You know enough about me by now to realise that I'm a "action first, thoughts later" kind of person, just trying to do some pre-emptive damage control
 
fair enough
:-D
 
@Leigh Yeh, that's what clones are for ;-)
 
user895378
Wat, how did I not know about --squash either!??!
 
@DaveRandom last thing I need at 5am on a saturday morning, realising i'm out of whisky, deciding it would be hilarious if I committed the rust-lang repo over master
 
9:15 PM
@DaveRandom actually.. the objmap->nodemap == DOM_NODESET case is the culprit...
can't return both a zval* and a XmlNodePtr :P
 
why not?
take a zval** as a paramter
 
how?
how does it fare with XmlNodePtr?
oh
no.. I need a zval** and a XmlNodePtr*
 
@FlorianMargaine You could return the xmlNodePtr*, but returning int is probably a better idea.
 
I return void and check if both zval** and xmlNodePtr* are NULL for the has_dimension
 
@FlorianMargaine If you return int (like the current has_dimension), has_dimension can just do return dom_nodelist_fetch_dimension(object, offset, NULL, NULL);
And read_dimension can do if (dom_nodelist_fetch_dimension(object, offset, &entry, &itemnode)) { /* ... */ }
 
9:25 PM
hm, indeed
 
/me is going to grab some food, bbiab
 
holy crap! the attacks have started!
 
:19083572 Isn't that something with Mario etc.?
 
the bullet was
 
@PaulCrovella That's a bummer.
 
9:33 PM
lol.
 
@ircmaxell yep, I mean… nice ^^
 
Our development director (in charge of interacting with donors and procuring donations) asked me to pull a bunch of data. She had no idea how to even describe the statistics she wanted, so I sat down and extracted the information from her. Made the query, exported the CSV, dropped it on her.
Now she wants it in graph form, so I show her Highcharts and ask her to pick the chart type and specify the columns
She had to get help doing that, now I'm making the charts.
They clearly demonstrate that ever since she started working here, we get less donations, and fewer pledges actually get paid in full.
So.... I expect her to ask me to make different charts when I come in tomorrow.
 
I really am trying hard not to troll the #hhvm room right now
 
freenode?
 
@ircmaxell Why?
 
9:41 PM
@Chris lol
 
because there are people talking about YAF and Falcon and why people are ditching them for Hack/HHVM
and I really want to reply "because they don't really understand why their shit was slow in the first place"
 
lol
Anyone jumping on board with Hack is pretty naive.
Early adoption at scale is not a good idea.
 
@LeviMorrison But they've got 'long term' support!1!
 
Hack is not "stable"; they have clearly stated they may change syntax.
 
@LeviMorrison looking for Saras talk on it though
 
9:42 PM
@Danack No, only for HHVM.
 
Where long term = time to make a baby.
They don't even have that for Hack?
 
Nope.
 
Amagad.
 
engine for scale. Not for speed
 
Notably their "callable" syntax is nasty and may change.
 
9:45 PM
their entire type system is questionable (IMHO)
and should change
It's like they had a bunch of people who said "Ohh, we could do this, wouldn't that be cool?!?!?", but they never stopped to ask if they should do it
 
But anyone adopting Hack for widespread usage or business critical pieces is premature unless paying developers to monitor it every day is cheaper than the costs of running under Zend.
 
> 17:46 <ThomasRedstone> I want to use Hack is the main reason, performance is a justification :-P I have one fellow shareholder, changing the programming language is not something he'd readily accept, but a change of technology that improves performance, he'll accept, it's an excuse
 
Hack is a new language.
Be explicit about that
Anyway, bed time for me (11:47 here)
Talk to you guys later!
 
later!
 
Commit and Push -- finally using Git at work. Welcome to the future.
We're checking out Phabricator as a project management system, going to demo it for a week and then choose between Atlassian and Phabricator. We get the former for free as a nonprofit, so it really is a battle of the featuresets.
 
9:50 PM
@DaveRandom there we go! I find it pretty now
 
I'm pulling for Phabricator only because of their sarcastic web site
 
10:01 PM
s/spaceship/hack - imgur.com/gallery/9odKE
 
[/goodnight]
 
@FlorianMargaine nice job :)
 
thanks
simple question (even though I know how to use them), what are TSRMLS_DC and TSRMLS_CC for?
 
3 hours ago, by NikiC
TSRMLS_CC == thread-safe resource manager local storage call with comma
and the D is for definition / declaration
if you made a function with no arguments, use TSRMLS_D, because you don't need the extra comma
 
I'm btw. in favor of debug builds not using the new native_tls branch, just to detect the stupid compiler errors… \cc @JoeWatkins
 
10:11 PM
@Leigh as I said, I know how to use them :P just dunno what they're for
thanks
 
@FlorianMargaine you know what a mutex is?
 
ok, they're for that ;D
 
in thread safe mode (--enable-maintainer-zts), they lock resources to stop multiple threads accessing at the same time
 
10:13 PM
nothing to do with mutex...
It's for thread local storage, nothing to do with locking
 
:x
I thought it was to prevent race conditions
 
@DaveRandom 5.6 backport ain't easy... :>
 
@Leigh it's for thread isolation
threads in PHP do not usually share resources. Only a couple exceptions
 
just cherry picked the commit... fixed the conflicts... pastie.margaine.com/07509879-c2b6-4d4b-a223-dac5f4f7a8fe
 
@NikiC I don't really get what it's for then
 
10:18 PM
hrm, I reached t99.php, now I'll have to overflow
 
@NikiC or mv {t,t0}*.php
 
is it just saying "I would like some storage space for my thread please" ?
"and make sure other threads can't have access to it, thanks"
 
I think it's more like saying "I want the resources in this thread"
 
@FlorianMargaine but, how do you do that?
@Leigh that's pretty much it
 
In bash?
 
10:22 PM
@FlorianMargaine use rename
works with patterns
rename t t0 t*.php
 
Doesn't the mv work?
 
no idea, I use rename when I want to bulk rename with patterns (it takes regex too)
 
nope, it doesn't :(
 
@FlorianMargaine The only thing I don't like is the ZVAL_COPY() in the fetch routine. At least it should be conditional on rv != NULL
 
@DaveRandom why?
 
10:26 PM
Because you don't need that data when you you do has_dimension, the copy is inefficient because you never use it
 
@NikiC use t+base36 number. that allows 1296 files…
 
indeed
@DaveRandom wait no, it might return false if this branch finds something
 
@FlorianMargaine why is rv a **?
 
will it work with this if it's not? github.com/Ralt/php-src/commit/…
testing
 
I'm having a little difficulty wrapping my head around this very simple thing
(It's been a long day)
 
10:34 PM
it's already a pointer... no reason it shouldn't work actually
yep, works
thanks @NikiC
 
btw @FlorianMargaine you should also be able to engineer it such that you can just pass NULLs directly to the two out args, rather that having to declare some unused stack vars (in has_dimension)
 
@DaveRandom yeah? wasn't sure if it wasn't just cleaner rather than having some stuff in fetch_dimension to take care of it
 
Not sure, it's tough to call because the fetch is only used in those two routines and almost certainly won't ever be used for anything else
It may be better to optimise it for those two specific use cases
I'm having a great deal of difficulty actually thinking about anything atm
 
I get that it's cleaner to pass just NULLs... but yeah, I'm not sure making the fetch routine uglier is the way to go
@DaveRandom go to bed then
 
Yeh I'm trying to unwind my brain by playing mindless JS games, but I think bed may be the only answer
I don't have to go to work tomorrow, but I do have to deal with a 3 year old all day, which is probably harder work
 
10:43 PM
it's a good answer, at least once a day
@DaveRandom enjoy :)
 
/me is out, nite @all
 
10:55 PM
@DaveRandom there we go github.com/php/php-src/pull/842
 
11:31 PM
earned 195 rep today… just not 200. lol.
I've repwhored too much today^^
 
11:45 PM
@rdlowrey For entertainment purposes only: github.com/symfony/symfony/issues/12007
 

« first day (1439 days earlier)      last day (3506 days later) »