Community
Participate
Working Groups
build 20021018 -In the Windows Explorer select a file and copy it to the clipboard -Delete the file -In the package view select a target ->The paste menu item is enabled even though the file on the clipboard does not exist. The paste operation results in an error. The paste menu item should be disabled in this scenario. Note: I just released a fix for bug 16129 that fixes this for the Navigator PasteAction.
fixed btw, code in PasteAction looks strange: if (resources[i].exists() == false) if (! resources[i].exists()) is used more commonly and certainly more readible
We've had that discussion before. Testing using ! is error prone. I've seen bugs that were hard to find because of an overlooked or missing !. ! is too easy on the eye, too easy to miss. Most people use ! without a space. I'm not sure the space helps any (readability making it less error prone). == false also isn't deleted accidentally, ! easily can be. This is one of my favorite subjects but it comes down to personal preference. I generally do my own thing but observe existing style when I hack in existing code. There's nothing worse than inconsistent coding style.
verified 20021113 BTW, I like if (! condition) {} better as well (can read more quickly). With the space there, you can't miss the exclamation mark. This is important when using proportional fonts.