Conversation started Jan 6, 2014 at 16:49.
Jan 6, 2014 16:49
@melak47 Wait, that is wrong. Where?
user3010322
I feel bad too: makes me feel like a stuck up snoody englishman. D:
@ThePhD No, that's the volatile. The (void) is to "use" the variable and shut the warning.
it doesn't like this: int n = std::ceil(time_limit / elapsed);
@ThePhD Nothing to do with bad ears, I suppose. More with an active brain (highly associative and ... perhaps a slight bit autistic? It takes me ages to figure out what people were actually trying to phonemize)
@melak47 WTF? Does their op/(duration, duration) return duration or some bullshit?
Jan 6, 2014 16:51
@ThePhD Yup me too. No problem with french accents here. German and, apparently, Russion accents are the worst though
@R.MartinhoFernandes looks like it :D I did a .count() on both :S
Yay for subverting nice libraries
Sep 13 '12 at 0:06, by sehe
@MooingDuck Time To Leave Grand Hotel
My physics teachers would have a stern word with them.
@sehe Neither. Ninja files are minimalistic by design. Any real work is meant to happen in the generator.
@melak47 What. The. Fuck.
That's so broken.
File a bug.
@R.MartinhoFernandes they don't listen to my bugs reports ._.
Jan 6, 2014 16:52
@R.MartinhoFernandes maybe plink @Luc
Xeo
Xeo
@melak47 Mail STL
@melak47 It's a common idiom for "discard this value".
@R.MartinhoFernandes so it's supposed to return a common_type of the duration...instead it returns a duration of common_type :D
Xeo
Xeo
@melak47 Common type of the representation-types
Can you dig the operator/ up?
Jan 6, 2014 16:55
yea
Xeo
Xeo
I'm curious how the fuck that's implemented, with which semantics.
user3010322
I can send the bug to STL.
wait no, it looks fine :/
user3010322
I have to wish him a happy new year still.
Then why is there a duration->double conversion going on on that line?
Xeo
Xeo
Jan 6, 2014 16:55
@melak47 So you fucked up?
I see no other possibility.
misread, the template density is too damn high :p
@LucDanton Wait, I think I misunderstood what you meant.
std::ratio<0x01,0x0989680> WTF
Jan 6, 2014 16:59
yeah, uh..that's also interesting
I guess their system clock has a weird period.
AFAICT I expected variables to ‘propagate’ across the graph (i.e. true dynamic scoping, no?), but it doesn’t do that. Only the variables of a particular build are used by the associated rule.
That would make my ‘workaround’ the proper way to do it then!
blargh
Wait. environment.h++:72 is times.reserve(n);
I had to shuffle some includes
Jan 6, 2014 17:04
Oh, I guess you added an include.
wheels and all
Right.
That seems quite the brokens.
so...do I just slap .count() on both and see what else breaks? :S
Yeah.
Wait, no.
Let me check.
Xeo
Xeo
dig up the impl of operator/ for durations please
Jan 6, 2014 17:05
@melak47 Yeah, don't do that.
time_limit and elapsed are not the same duration type.
wtf is up with their indentation
Xeo
Xeo
huh, looks correct
is elapsed no a duration or something?
@Xeo One is FloatDuration<Clock>, the other Duration<Clock>.
@rightfold using rightfold = clang-format-bot;
user1804599
Jan 6, 2014 17:08
using rightfold = tab-remover;
(Almost the same, but the former is forced to have rep as double)
Xeo
Xeo
then I don't get it
@Xeo Doesn’t look too SFINAE-friendly, if that’s ever a concern.
user3010322
namespace Prison {
     const static auto unusable = rightfold{};
}
user3010322
D:<
Jan 6, 2014 17:10
@melak47 Can you struct{}_=decltype(time_limit){}; and struct{}_=decltype(elapsed){};?
needs {}
Right.
"expected primary-expression" is not too helpful.
nonius/environment.h++(72): error C2440: 'initializing' : cannot convert from 'std::chrono::duration<double,std::ratio<0x01,0x0989680>>' to 'nonius::estimate_clock_cost::<unnamed-type-_>'
thats time_limit
and
error C2440: 'initializing' : cannot convert from 'std::chrono::duration<std::chrono::system_clock::rep,std::chrono::system_clock:‌​:period>' to 'nonius::estimate_clock_cost::<unnamed-type-_>'
Jan 6, 2014 17:13
Ugh, fuck you MSVC, thanks a lot.
Does it expand what std::chrono::system_clock::rep and std::chrono::system_clock::period are in the following text?
Xeo
Xeo
does that matter? both operands are durations
it doesn't :/
Xeo
Xeo
so it should select the right overload
Well, whatever. What of decltype(time_limit / elapsed)?
Xeo
Xeo
and yield an arithmetic type
Jan 6, 2014 17:15
error C2446: ':' : no conversion from 'std::chrono::duration<std::chrono::system_clock::rep,std::chrono::system_clock::period>' to 'double'
1> No user-defined-conversion operator available that can perform this conversion, or the operator cannot be called
1> D:\Files\Dropbox\c++\vc12\ConsoleApplication2\ConsoleApplication2\deps\nonius/environment.h++(72) : see reference to class template instantiation 'std::common_type<double,Duration>' being compiled
1> with
1> [
Xeo
Xeo
just wtf
What's in type_traits(1446)?
template<class _Ty0,
	class _Ty1>
	struct common_type<_Ty0, _Ty1>
	{	// type is common type of _Ty0 and _Ty1 for two arguments
	typedef typename decay<
		decltype(_Always_false<_Ty0>::value
			? _STD declval<_Ty0>()
			: _STD declval<_Ty1>())
	>::type type;
	};
Jan 6, 2014 17:16
the ":"
lol?
haha, no way.
What are the arguments to the instantiation?
I wanna laugh, too
@LucDanton double, Direction, see log above
9 mins ago, by Luc Danton
@Xeo Doesn’t look too SFINAE-friendly, if that’s ever a concern.
Jan 6, 2014 17:17
@melak47 What?
> see reference to class template instantiation 'std::common_type<double,Duration>' being compiled
gief cookie
@melak47 It tries the first overload, and it hard errors there instead of SFINAE.
Jan 6, 2014 17:19
brb making tea, to go with cookie
evening lounge
@LucDanton :D
@melak47 IOW, <chrono> broken on MSVC.
just stop this silly SFINAE nonsense, it's all your fault
@melak47 What? The SFINAE is in the stdlib.
Failure is not an excuse?
;D
Jan 6, 2014 17:21
It's the SFINAE from the stdlib that is broken.
I have no SFINAE at all in nonius.
The only templates I have are straightforward for letting you pick Clock, IIRC.
I'm curious how this got past untested.
user3010322
Music too strong.
this isn't even the CTP
it could be even brokener there :D
Any test of this feature would bring the error.
user3010322
@melak47 "RTM Quality" ._.
@R.MartinhoFernandes Perhaps the question is the answer here.
Jan 6, 2014 17:23
@R.MartinhoFernandes ain't nobody got time for that
user3010322
Now you guys know why I want to join the STL team?
It's really the kind of bug that only passes by if they just slap it together and don't even bother testing.
fuck, they started accepting my answers when I'm way over rep cap. they should've done it yesterday @#$%$$#@
user3010322
We need a Lounger on the inside
user3010322
Who can fix bugs ASAP.
Jan 6, 2014 17:23
@ThePhD Because you would fit right in? SCNR.
@BartoszKP accepts go over rep cap
user3010322
@R.MartinhoFernandes q__q
@ThePhD Wrong approach.
user3010322
my heart.
I don't care about fixing this bug.
user1804599
Jan 6, 2014 17:24
I hope tomorrow’s weather will be as nice as today’s.
There is no test for this feature in the stdlib test suite.
@nightcracker yeah, I know. but yesterday I didn't have a feeling of fulfilment :( and today I already have
Letting you mix and match durations of different types is pretty much the raison d'être of <chrono>.
So yes, I think this is quite embarrassing.
 
Conversation ended Jan 6, 2014 at 17:25.