A *.userdata=struct(...) direct assignment erases the parent handle. Work-aroun

Reported by Samuel GOUGEON (@sgougeon)

-- Bug description --


A *.userdata=struct(...) direct assignment erases the parent handle.
Work-around: set(handle,"userdata",structure) works well.
May be a parser issue instead of a Graphics-related one.


-- Scilab error message --


None. The variable containing the handle is destroyed instead of being updated.


-- How to reproduce the bug --


clf
plot2d()
a=gca();
s=struct("text","Hello","num",rand(2,2))
a.userdata=s
a=gca();
set(a,"userdata",s)

assigned to @bruno.jofret

added Bugzilla Graphic plotting Major PC master GIT version [master branch] labels

By Samuel GOUGEON (@sgougeon)

set(a,"userdata",s)

Then, it is not possible to directly update the fields of the userdata structure. No error nor warning is issued, but the update is not done: a.userdata.text="Bonjour" leaves the "Hello" value Fields are well adressed in read mode, but not in write mode: a.userdata.num(3) is correctly adressed for display.

Samuel

By Sylvestre LEDRU

Hmm, I don't see the problem:

-->clf

-->plot2d()

-->a=gca();

-->s=struct("text","Hello","num",rand(2,2)) s =

text: "Hello" num: [2x2 constant]

-->a.userdata=s a =

userdata: [1x1 struct]

-->a.userdata.text="Bonjour" a =

userdata: [1x1 struct]

-->a.userdata.text ans =

Bonjour

By Calixte DENIZET

Hello Sylvestre,

An other way to show you the problem:

s=struct("text","Hello","num",rand(2,2)); a=gca();typeof(a) // a is a handle a.username=s;typeof(a) // a is a struct

Calixte

By Allan CORNET

this trouble was already in 4.1.2

Allan

By Bruno JOFRET (@bruno.jofret)

a=gca(); typeof(a) // a is a handle a.user_data = struct(); typeof(a) // a is a struct

This works on every kind of struct insert within a handle... -->a = gca();

-->a.mark_mode = struct() a =

mark_mode: [0x0 struct]

By Bruno JOFRET (@bruno.jofret)

Fixed in commit : 7420251cd6fab797db0f602c79c21bfacb8404ba

https://gitlab.com/scilab/legacy_codereview/-/wikis/changes/2511

closed

reopened

By Sylvestre LEDRU

Fixed: 513a0cef

closed