| Time |
Nick |
Message |
| 01:27 |
|
DI3HARD139 joined #minetest-dev |
| 02:19 |
|
ANAND joined #minetest-dev |
| 02:34 |
|
TC01 joined #minetest-dev |
| 03:05 |
|
ANAND joined #minetest-dev |
| 03:15 |
|
AndroBuilder joined #minetest-dev |
| 03:37 |
|
AndroBuilder joined #minetest-dev |
| 03:39 |
|
AndroBuilder_ joined #minetest-dev |
| 03:48 |
|
AndroBuilder joined #minetest-dev |
| 03:54 |
|
AndroBuilder joined #minetest-dev |
| 04:13 |
|
AndroBuilder joined #minetest-dev |
| 04:26 |
|
Darcidride_ joined #minetest-dev |
| 04:26 |
|
RichardTheTurd joined #minetest-dev |
| 04:44 |
|
Darcidride joined #minetest-dev |
| 05:30 |
|
RichardTheTurd joined #minetest-dev |
| 06:49 |
|
chon joined #minetest-dev |
| 08:47 |
|
DI3HARD139 joined #minetest-dev |
| 09:11 |
|
Wuzzy joined #minetest-dev |
| 10:36 |
|
fwhcat joined #minetest-dev |
| 11:17 |
|
Fixer joined #minetest-dev |
| 11:28 |
|
Krock joined #minetest-dev |
| 11:46 |
|
Raven262 joined #minetest-dev |
| 11:52 |
|
Fixer joined #minetest-dev |
| 12:07 |
|
longerstaff13 joined #minetest-dev |
| 12:22 |
|
lumberJ joined #minetest-dev |
| 12:29 |
|
Kalvo joined #minetest-dev |
| 12:31 |
Kalvo |
Hey everyone quick question. Does node metadata get destroyed when a node gets destroyed? (Or when using set_node()) |
| 12:33 |
rubenwardy |
yes with set_node |
| 12:33 |
rubenwardy |
which is how nodes are dug under the hood |
| 12:33 |
rubenwardy |
swap_node and LVMs don't do this, and don't run callbacks either |
| 12:52 |
|
Foz joined #minetest-dev |
| 13:37 |
|
antims joined #minetest-dev |
| 13:50 |
|
Beton joined #minetest-dev |
| 13:55 |
|
troller joined #minetest-dev |
| 14:22 |
|
proller joined #minetest-dev |
| 14:31 |
|
proller joined #minetest-dev |
| 14:33 |
|
troller joined #minetest-dev |
| 15:22 |
|
proller__ joined #minetest-dev |
| 15:39 |
|
twoelk joined #minetest-dev |
| 16:08 |
|
Player-2 joined #minetest-dev |
| 16:16 |
|
Darcidride_ joined #minetest-dev |
| 16:18 |
|
Fixer joined #minetest-dev |
| 16:29 |
|
Krock joined #minetest-dev |
| 16:40 |
|
Krock joined #minetest-dev |
| 16:49 |
|
lumberJ joined #minetest-dev |
| 17:03 |
|
longerstaff13 joined #minetest-dev |
| 17:19 |
Krock |
rubenwardy, do you have any opinion/reply for this comment? https://github.com/minetest/minetest/pull/7214/files#r182687246 If not, I will merge the PR this evening |
| 17:19 |
rubenwardy |
it will never be null |
| 17:19 |
rubenwardy |
hmmm |
| 17:20 |
Krock |
it can be null when the reference is kept in Lua |
| 17:20 |
rubenwardy |
issue is, it should be documented if it ever turns null |
| 17:20 |
rubenwardy |
no, all the providers use pointers |
| 17:20 |
rubenwardy |
so it'll just be a garbage pointer |
| 17:20 |
rubenwardy |
it's probably best to change that to return 0; to futue proof it |
| 17:20 |
rubenwardy |
needs to be documented as well |
| 17:21 |
|
Foz joined #minetest-dev |
| 17:21 |
Krock |
for the sake of completeness `nil` is appropriate there. Do you have time to update it or shall I take care of it? |
| 17:24 |
rubenwardy |
won't have time today |
| 17:26 |
|
Foz joined #minetest-dev |
| 17:32 |
|
rubenwardy joined #minetest-dev |
| 17:45 |
Krock |
I see. The NodeMetaRef::getmeta function is safe and NULL return values are possible there, whereas it looks quite bad with other MetaData types |
| 17:49 |
|
Raven262 joined #minetest-dev |
| 18:04 |
|
proller__ joined #minetest-dev |
| 18:28 |
|
Foz joined #minetest-dev |
| 18:42 |
|
chon joined #minetest-dev |
| 18:50 |
|
rubenwardy joined #minetest-dev |
| 19:03 |
|
Foz joined #minetest-dev |
| 19:05 |
|
Foz joined #minetest-dev |
| 19:14 |
|
proller joined #minetest-dev |
| 20:16 |
|
Foz joined #minetest-dev |
| 20:30 |
|
nerzhul joined #minetest-dev |
| 21:27 |
|
Gael-de-Sailly joined #minetest-dev |