View Issue Details

IDProjectCategoryView StatusLast Update
0006416Spring engineLuapublic2020-08-11 22:23
ReporterKrogoth Assigned Tohokomoko  
PrioritynormalSeverityfeatureReproducibilityalways
Status closedResolutionwon't fix 
Product Version104.0 
Summary0006416: rawset, rawget, rawequal
Descriptionrawset, rawget, rawequal appear to be nil in synced context. (defined in others)
Is it intended?
rawset is necessary to wrap some things up
Steps To ReproduceSpring.Echo(rawset) in a gadget
TagsNo tags attached.
Checked infolog.txt for ErrorsIrrelevant

Activities

hokomoko

2020-08-10 21:24

developer   ~0020505

my bet is that you're using custom gadget handler code where this line doesn't exist https://github.com/spring/spring/blob/30862626214bd263b1c4489bb197ef1c3dbc0738/cont/base/springcontent/LuaGadgets/system.lua#L104

Krogoth

2020-08-10 22:16

reporter   ~0020506

can you reproduce to verify
for me, its nil everywhere, including ba9.46

Kloot

2020-08-11 12:08

developer   ~0020507

intended: https://github.com/spring/spring/blob/develop/rts/Lua/LuaHandleSynced.cpp#L390

Krogoth

2020-08-11 18:21

reporter   ~0020508

i noticed "//FIXME not unsafe anymore since split?"
implying, it's possibly safe now?
if it's not certain, can u expose it via different identifier with prefix to test it out, and what kind of usage exactly might turn it dangerous

Krogoth

2020-08-11 18:22

reporter   ~0020509

oops, forgot question mark
what kind of usage exactly might turn it dangerous ?

hokomoko

2020-08-11 22:23

developer   ~0020510

As far as I understood, Krogoth wants this in order to make tables with read only fields that only change through function.
Personally I don't see it as an important enough reason to put risky stuff in synced land.

Issue History

Date Modified Username Field Change
2020-08-10 18:26 Krogoth New Issue
2020-08-10 21:24 hokomoko Note Added: 0020505
2020-08-10 21:24 hokomoko Assigned To => hokomoko
2020-08-10 21:24 hokomoko Status new => feedback
2020-08-10 22:16 Krogoth Note Added: 0020506
2020-08-10 22:16 Krogoth Status feedback => assigned
2020-08-11 12:08 Kloot Note Added: 0020507
2020-08-11 18:21 Krogoth Note Added: 0020508
2020-08-11 18:22 Krogoth Note Added: 0020509
2020-08-11 22:23 hokomoko Status assigned => closed
2020-08-11 22:23 hokomoko Resolution open => won't fix
2020-08-11 22:23 hokomoko Note Added: 0020510