El 06/10/2008 07:31 AM, Pau Arumí Albó escribió:
El dt 10 de 06 de 2008 a les 06:53 -0300, en/na Natanael Olaiz va
escriure:
El 06/10/2008 06:03 AM, Pau Arumí Albó escribió:
Well, in the processing adapter I putted a check line to don't store the new attributes if are not given, but any check on the load... since it works good reading a file with or without the new attributes. I don't think it would affect other uses (i.e. the backwards compatibility wouldn't be affected). The major problem as it's now is if you load a network with the attributes, it will fill the _processingsBoxesAttributes map. So, I send here a new patch for Network.cxx that clear it every time you do a load, to avoid fill all the memory if you load so much times without clear it.

Do you think it's a problem that _processingsBoxesAttributes could be non-empty without using it?

Er yes could lead to potential problems. We don't want duplicated
information and the original info sits in the canvas, so let's use it
when loading and saving and keep it clean the rest of the time.

David: help! :)
The only solution that occurs to me is using a toggle attribute like setPasteMode. But since this wouldn't be used only for paste but loading files... should I add another?

I don't like it... but it's the only idea that comes to my mind to solve that...


Regards,
Natanael.

_______________________________________________
Clam-devel mailing list
[email protected]
https://llistes.projectes.lafarga.org/cgi-bin/mailman/listinfo/clam-devel

Reply via email to