You must be registered to post a reply. Click here to register.
T O P I C R E V I E W
fredizzimo
Posted - May 17 2009 : 06:00:16 AM In addition to the things in the "Positive confirmation only" topic, this is my other major annoyance with visual assist, and it also destroys my programming flow. This is also a very old case, case=7802, nearly two years old.
I encounter this almost daily, as our mock object library works like this. I type Expect().Once().On(object).Function(&Class:Function).With(5). Everything works fine it flows nicely, I added some simlpler macros so visual assist doesn't get confused. But once I come to the function part it all breaks down, I most likely type &Class:F and press tab. It expands to &Class:Function(), and puts my cursor between the parenthesis, since its a function taking an int argument. Now to undo this effect I have to press two different keys(arrow+del+del, arrow+backspace+backspace, backspace+del or del+backspace), which is very counter productive.
I don't buy the argument that you sometimes might want to take the adress of the return value. First of all a return value is an rvalue, c++ standard 3.10-Temporaries are rvalues (See also lifetime). Secondly you cannot take an address of an rvalue, c++ standard - 5.3.1/2 The operand of & must be an lvalue. So the code is in fact not valid c++, even if some compilers might accept it.
Secondly what's wrong with the code expanding to &Class:Function at first, then when you press ( it shows the function arguments. All of this according to the positive flow and positive confirmations as I have been talking about.
4 L A T E S T R E P L I E S (Newest First)
support
Posted - Aug 22 2009 : 7:17:25 PM There was an additional problem with case=7802 when namespaces were present. This is fixed in build 1731.
sean
Posted - Jun 12 2009 : 9:03:33 PM case=7802 is fixed in build 1727
fredizzimo
Posted - May 18 2009 : 4:09:45 PM Thank you, that's great to hear :)
accord
Posted - May 18 2009 : 3:24:33 PM case=7802 has been fixed, and the fix should appear in the next or next few builds.