Time |
Nick |
Message |
00:12 |
|
SFENCE joined #luanti-dev |
00:18 |
MTDiscord |
<luatic> btw how do we feel about std::size_t vs size_t |
00:18 |
MTDiscord |
<luatic> supposedly the former is more lawful |
00:33 |
|
exoticalexo joined #luanti-dev |
00:38 |
|
exoticalexo joined #luanti-dev |
00:45 |
|
SFENCE joined #luanti-dev |
01:00 |
|
exoticalexo joined #luanti-dev |
01:20 |
|
exoticalexo joined #luanti-dev |
01:22 |
|
exoticalexo joined #luanti-dev |
01:36 |
|
SFENCE joined #luanti-dev |
02:04 |
|
exoticalexo joined #luanti-dev |
02:28 |
|
SFENCE joined #luanti-dev |
03:00 |
|
exoticalexo joined #luanti-dev |
03:01 |
|
exoticalexo19 joined #luanti-dev |
03:01 |
|
SpaceManiac joined #luanti-dev |
03:03 |
|
exoticalexo16 joined #luanti-dev |
03:18 |
|
SFENCE joined #luanti-dev |
03:52 |
|
SFENCE joined #luanti-dev |
04:00 |
|
MTDiscord joined #luanti-dev |
04:31 |
|
SFENCE joined #luanti-dev |
04:44 |
|
TheCoffeMaker joined #luanti-dev |
05:05 |
|
SFENCE joined #luanti-dev |
05:38 |
|
SFENCE joined #luanti-dev |
06:11 |
|
SFENCE joined #luanti-dev |
07:18 |
sfan5 |
I use the latter |
07:20 |
|
SFENCE joined #luanti-dev |
07:21 |
|
pmp-p joined #luanti-dev |
10:50 |
[MatrxMT] |
<Zughy> sfan5: why possible close on #14643 ? |
10:50 |
ShadowBot |
https://github.com/luanti-org/luanti/issues/14643 -- Add Active Object Spatial Cache by ExeVirus |
10:53 |
sfan5 |
mutually exclusive with luatic's PR |
11:41 |
MTDiscord |
<josiah_wi> If you want the most lawful size type for working with a standard container, it's the size_type attribute of that container. 🙂 |
12:04 |
|
fgaz joined #luanti-dev |
12:04 |
|
HamAdams joined #luanti-dev |
13:01 |
|
behalebabo joined #luanti-dev |
13:13 |
|
BuckarooBanzai joined #luanti-dev |
13:15 |
|
[_ joined #luanti-dev |
13:16 |
|
pmp-p_ joined #luanti-dev |
14:10 |
|
SFENCE joined #luanti-dev |
15:25 |
|
SFENCE joined #luanti-dev |
15:34 |
|
SFENCE joined #luanti-dev |
15:37 |
|
fluxionary joined #luanti-dev |
15:43 |
|
SFENCE joined #luanti-dev |
15:47 |
|
SFENCE joined #luanti-dev |
16:20 |
|
SFENCE joined #luanti-dev |
16:35 |
|
SFENCE joined #luanti-dev |
16:35 |
|
Desour joined #luanti-dev |
16:49 |
|
SFENCE joined #luanti-dev |
16:54 |
Desour |
I definitely prefer size_t. std::size_t (or even the container's typedef) is just too verbose |
17:09 |
|
fluxionary joined #luanti-dev |
17:11 |
|
SFENCE joined #luanti-dev |
17:22 |
|
m42uko joined #luanti-dev |
17:34 |
[MatrxMT] |
<grorp> merging #15972 in 5 min |
17:34 |
ShadowBot |
https://github.com/luanti-org/luanti/issues/15972 -- lua_api.md: Mapblock-related and misc improvements by grorp |
17:44 |
|
SFENCE joined #luanti-dev |
17:50 |
|
SFENCE joined #luanti-dev |
18:01 |
|
SFENCE joined #luanti-dev |
18:25 |
|
SFENCE joined #luanti-dev |
18:45 |
MTDiscord |
<herowl> Just using std::size_t; and you've got your lawful usage... |
18:47 |
MTDiscord |
<herowl> Or import a container typedef |
18:47 |
MTDiscord |
<herowl> Potentially abuse auto |
18:48 |
MTDiscord |
<herowl> Anyway, I really don't like size_t in loops, especially with non-trivial increment/decrement logic. |
18:48 |
MTDiscord |
<herowl> (unsigned) |
19:34 |
|
SFENCE joined #luanti-dev |
20:05 |
|
SFENCE joined #luanti-dev |
20:39 |
|
SFENCE joined #luanti-dev |
21:04 |
|
SFENCE joined #luanti-dev |
21:38 |
|
SFENCE joined #luanti-dev |
22:06 |
|
SFENCE joined #luanti-dev |
22:33 |
|
panwolfram joined #luanti-dev |
22:40 |
|
SFENCE joined #luanti-dev |
23:26 |
|
SFENCE joined #luanti-dev |
23:35 |
MTDiscord |
<luatic> it looks like our operator[] on vector(2|3)d is UB. |
23:35 |
MTDiscord |
<luatic> it also looks like compilers can't optimize a switch that effectively does array indexing, even if the struct is laid out like an array. |
23:37 |
MTDiscord |
<luatic> in practice the UB is hopefully not a problem but who knows. |