2020-10-01 03:11 CEST

View Issue Details Jump to Notes ]
IDProjectCategoryView StatusLast Update
0006416Spring engineLuapublic2020-08-11 22:23
ReporterKrogoth 
Assigned Tohokomoko 
PrioritynormalSeverityfeatureReproducibilityalways
StatusclosedResolutionwon't fix 
Product Version104.0 
Target VersionFixed in Version 
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
Attached Files

-Relationships
+Relationships

-Notes

~0020505

hokomoko (developer)

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

~0020506

Krogoth (reporter)

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

~0020507

Kloot (developer)

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

~0020508

Krogoth (reporter)

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

~0020509

Krogoth (reporter)

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

~0020510

hokomoko (developer)

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.
+Notes

-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
+Issue History