[Issue 1886] Inserting into an AA of structs with opAssign results in ArrayBoundsError
https://issues.dlang.org/show_bug.cgi?id=1886 Andrei Alexandrescu and...@erdani.com changed: What|Removed |Added Version|2.023 |D2 --
[Issue 1886] Inserting into an AA of structs with opAssign results in ArrayBoundsError
http://d.puremagic.com/issues/show_bug.cgi?id=1886 Don clugd...@yahoo.com.au changed: What|Removed |Added Status|NEW |RESOLVED Resolution||DUPLICATE --- Comment #6 from Don clugd...@yahoo.com.au 2010-10-19 15:13:39 PDT --- The patch for bug 2451 fixes this. Clearly a duplicate. *** This issue has been marked as a duplicate of issue 2451 *** -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email --- You are receiving this mail because: ---
[Issue 1886] Inserting into an AA of structs with opAssign results in ArrayBoundsError
http://d.puremagic.com/issues/show_bug.cgi?id=1886 Don clugd...@yahoo.com.au changed: What|Removed |Added CC||clugd...@yahoo.com.au --- Comment #5 from Don clugd...@yahoo.com.au 2010-05-25 13:27:52 PDT --- Duplicate of bug 2451. However there is some useful information in the comments for this bug. -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email --- You are receiving this mail because: ---
[Issue 1886] Inserting into an AA of structs with opAssign results in ArrayBoundsError
http://d.puremagic.com/issues/show_bug.cgi?id=1886
Michael Rynn y0uf00...@gmail.com changed:
What|Removed |Added
CC||y0uf00...@gmail.com
--- Comment #2 from Michael Rynn y0uf00...@gmail.com 2009-12-16 12:59:21 PST
---
Its not adding just opAssign. Having post-blit in a struct can do it too.
Documentation on struct in Associative Arrays in D2 is out of date. Probably
because it mostly does not work. To return a pointer to the struct have to take
address of this
struct pblit {
int x;
pblit* opAssign(ref const pblit S)
{
this.x = S.x;
}
this(this)
{
}
~this()
{
}
}
void main(char[][] args)
{
pblit[string] map;
string[pblit] rmap;
pblit val;
val.x = 12;
rmap[val] = test; // this works
map[test] = val;// oops! range error
}
--
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
[Issue 1886] Inserting into an AA of structs with opAssign results in ArrayBoundsError
http://d.puremagic.com/issues/show_bug.cgi?id=1886 --- Comment #4 from Michael Rynn y0uf00...@gmail.com 2009-12-16 19:10:53 PST --- Sorry to split into 3 bits. When I press tab the focus goes to the submit button. I stepped thru the x86 code generated in the debugger using Visual Studio 2008 after conversion with cv2pdb -- great tool. In the noblit case :- The code calls __aaGet the arguments are the key, typeinfo...@__init, and address of the map structure. An address? is returned in EAX, which is checked for null, The value of nval is directly blitted to the returned address. __aaGet does the checking if the key points to existing value, and allocates memory for new value as required. In the postblit case the generated code seems to miss the point entirely. The code initially makes a temporary copy of val. and calls the postblit constructor, which calls this(this). -- @postblit__cpctor -- postb...@__postblit. Why is a prior temporary copy required, when one hopes that its the map that is going to provide the final destination? From the temporary copy, it then pushes the bits value of the temporary (not the address!), then the key, then typeinfo...@__init, and address of map. The code then calls __aaGetRvalue ( not __aaGet ) __aaGetRvalue gives up very quickly, failing its first check. It pushes registers, and then checks the address in [esp+18h]. This being null, it pop-exits with null as the result in eax. Obviously __aaGetRvalue is expecting something else to be setup. Presumably if __aaGetRvalue actually worked, it would just reblit the value in the temporary to the memory it allocated or found, without calling postblit again. I would like to see ideally. No temporary copy made initially. The map lookup able to allocate or find memory for the key. The address returned. The copy made and then post-blit called. Failing that , please fix up __aaGetRvalue so it does the right thing. -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email --- You are receiving this mail because: ---
