[Octave-bug-tracker] [bug #32343] last plot resets all other curve labels

[David Bateman]

Follow-up Comment #5, bug #32343 (project octave):

John, 

The listener functions are needed to allow things like

plot(1:2, "-r")
h = legend("I am red")
set (h, "location", "east")

to work as expected. I believe the issue is rather that in __plt__ we aren't
taking into account any existing legends when called. So the fix is  to
correctly initialize hlgnd and tlgnd in __plt__. Ben's original fix worked for
#32022 because he explicitly set the correct axis handle and effectively
removed the call to fliplr.

Adding fliplr again works for #32343 because the second call to __plt__ and
the embedded second call to the legend function makes the "2" legend key be
assigned to the last key set. However this won't work for a case like

plot (1:3,"-r;1;");hold on; plot(2*(1:3),"-g;2;",3*(1:3),"-r;3;")

as the second and third labels will be reversed. Note that flipud and fliplr
shouldn't be needed at all.

I believe the attached changeset fixes this issue, but I haven't pushed it
yet.

D.


(file #22608)
    _______________________________________________________

Additional Item Attachment:

File name: patch.legend                   Size:8 KB


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?32343>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/