Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug: RMBClick while in VEF modes #120

Closed
necros0 opened this issue Jul 30, 2012 · 36 comments
Closed

Bug: RMBClick while in VEF modes #120

necros0 opened this issue Jul 30, 2012 · 36 comments
Assignees
Labels
Type:Bug Errors and problems

Comments

@necros0
Copy link
Contributor

necros0 commented Jul 30, 2012

If you right-click while in one of the VEF manip modes a new brush seems to be created at the click point 1 unit in dimensions, however this new brush cannot be interacted with at all.
Further, it appears to be selected but clicking on other brushes does not deselect it.
RMBClicking again will clear this 'ghost' brush.

@ghost ghost assigned kduske Aug 4, 2012
@kduske
Copy link
Collaborator

kduske commented Aug 7, 2012

I cannot reproduce this. When I do it, everything is deselected and the editor leaves VEF mode. If I right drag while in VEF mode, the editor leaves VEF mode and I can create a brush as usual.

@necros0
Copy link
Contributor Author

necros0 commented Aug 7, 2012

This is the exact steps:

  • Create a brush
  • While brush is selected, press v
  • RMBClick anywhere in 3d View

Results in something like this. You can see the 8x8x8 "brush" that is left over. You can select all normal brushes and perform operations on them as normal (including going into VEF modes).

@kduske
Copy link
Collaborator

kduske commented Aug 7, 2012

Alright, I can reproduce this on Windows, too.

@kduske
Copy link
Collaborator

kduske commented Aug 12, 2012

This should be fixed in TrenchBroom_Win32_20120812_1311, please confirm.

@necros0
Copy link
Contributor Author

necros0 commented Aug 12, 2012

Edit: I spoke too soon, sometimes when creating a brush, the program will just quit silently.

Also, repeating the steps in this post results in the program quitting with no errors.

@kduske
Copy link
Collaborator

kduske commented Aug 12, 2012

Strange - I had that happen last week and I thought I fixed it. Unfortunately it's very hard to debug because it only happens in the Release build on Windows. I'll work on that now.

@necros0
Copy link
Contributor Author

necros0 commented Aug 12, 2012

It seems to happen when, during the RMBClick, you drag the mouse enough that the single unit brush would become more than 1 unit.

@kduske
Copy link
Collaborator

kduske commented Aug 12, 2012

Yeah I know. It is somehow related to the rendering of the size helper graphic while right dragging.

@kduske
Copy link
Collaborator

kduske commented Aug 12, 2012

Could you please try TrenchBroom_Win32_20120812_1929 and report back? That version definitely does not crash for me when creating a brush by right-dragging. The Vertex-Mode problem you described earlier doesn't exist either.

@necros0
Copy link
Contributor Author

necros0 commented Aug 12, 2012

I believe this thread's specific bug is fixed. If you carefully Rightclick in VEF mode, a new brush is created and VEF mode is disabled.

It was just the RMBDrag crash causing the initial bug (which is still happening in build 1929).

@kduske
Copy link
Collaborator

kduske commented Aug 12, 2012

I don't understand it. It's working fine for me now. I'll see if I can reproduce this.

@necros0
Copy link
Contributor Author

necros0 commented Aug 12, 2012

Feel free to send me some executables to test changes.

@kduske
Copy link
Collaborator

kduske commented Aug 12, 2012

I'm trying to reproduce it like this:

  • start editor
  • in the new map, right drag to create brush

Does it crash when you do that?

@kduske
Copy link
Collaborator

kduske commented Aug 12, 2012

Also, which version of Windows are you using?

@necros0
Copy link
Contributor Author

necros0 commented Aug 12, 2012

Windows 7 Professional, SP1. I usually install all critical updates and most of the optional ones if they seem relevant.

reply to this
Yes, this is all it takes. It happens at any time though, just dragging such that the created brush becomes larger than 1 unit cube will cause the crash.

@kduske
Copy link
Collaborator

kduske commented Aug 12, 2012

Hmm, I have tested it on both Windows 7 64 bit and Windows XP 32 bit. I'm currently out of ideas... I had the bug a couple of days ago and tracked it down to a specific location, and then it disappeared. Something fishy is going on... I'll try it on another machine and see if I can reproduce it there.

@kduske
Copy link
Collaborator

kduske commented Aug 12, 2012

Please try TrenchBroom_Win32_20120812_2313.

@kduske
Copy link
Collaborator

kduske commented Aug 12, 2012

Okay, I have an idea how to track this down, but this will be quite annoying for you. If you could try each version in dropbox in reverse order and tell which is the latest version that does not exhibit this error, this might help me find the problem...

@necros0
Copy link
Contributor Author

necros0 commented Aug 12, 2012

The last version that worked correctly for me was TrenchBroom_Win32_20120804_1047.zip
The problem started with TrenchBroom_Win32_20120812_1311.zip.

@kduske
Copy link
Collaborator

kduske commented Aug 12, 2012

Aha, that should help me find it. I'll compare the versions tomorrow.

@necros0
Copy link
Contributor Author

necros0 commented Aug 13, 2012

Of all the versions (1831, 2135, 1213) uploaded today, TrenchBroom_Win32_20120809_1831 no longer crashes when creating brushes and also has the VEF mode fix.

@kduske
Copy link
Collaborator

kduske commented Aug 14, 2012

What about the later versions? They crash again?

@necros0
Copy link
Contributor Author

necros0 commented Aug 14, 2012

Later versions?
Maybe it was just dropbox goofing around but it redownloaded

  • TrenchBroom_Win32_20120809_1831
  • TrenchBroom_Win32_20120807_2135
  • TrenchBroom_Win32_20120810_1213

on the 13th. (they have newer data created time stamps)
I thought that was you experimenting with the code.
If it was just dropbox redownloading for no reason, then ignore that last post.

@kduske
Copy link
Collaborator

kduske commented Aug 14, 2012

It must have been dropbox, because I will never reuse a build number.

@kduske
Copy link
Collaborator

kduske commented Aug 14, 2012

Please try TrenchBroom_Win32_20120814_2111.

@necros0
Copy link
Contributor Author

necros0 commented Aug 14, 2012

TrenchBroom_Win32_20120814_2111: Sorry man, still there. :(

@necros0
Copy link
Contributor Author

necros0 commented Aug 14, 2012

I think I missed something. Did you upload earlier versions on dropbox yesterday?

Because in this post I mentioned that TrenchBroom_Win32_20120804_1047was the latest version that stopped working.

But with the 3 files that were uploaded yesterday, this is now how it looks:

  • TrenchBroom_Win32_20120804_1047 - FINE
  • TrenchBroom_Win32_20120807_2135 - FINE
  • TrenchBroom_Win32_20120809_1831 - FINE
  • TrenchBroom_Win32_20120810_1213 - CRASHES
  • TrenchBroom_Win32_20120812_1311 - CRASHES

I'm not sure if I got mixed up with all the versions (It would be nice if the build number always incremented) or if dropbox didn't upload some files until later.

@kduske
Copy link
Collaborator

kduske commented Aug 14, 2012

It's possible that dropbox got mixed up on the latest version. The build number is always different: TrenchBroom_Win32_YYYYMMDD_HHmm.zip where YYYY = year, MM = month, DD = day of month, HH = hour in 24-hour format, mm = minutes.

@kduske
Copy link
Collaborator

kduske commented Aug 15, 2012

Hi, I have added lots of debugging output to TrenchBroom_Win32_20120815_1357. Please try it and send me the log file. If you have skype or some other instant messenger, we could debug this much more quickly though.

@necros0
Copy link
Contributor Author

necros0 commented Aug 15, 2012

Strange, TrenchBroom_Win32_20120815_1357 no longer crashes when making brushes. Log
I don't have skype (or a mic) but I could install an instant messenger program if you let me know which one you want to use.

@kduske
Copy link
Collaborator

kduske commented Aug 15, 2012

Strange, TrenchBroom_Win32_20120815_1357 no longer crashes when making brushes.

Jesus, I bet the next build will crash again.

I don't have skype (or a mic) but I could install an instant messenger program if you let me know which one you want to use.

I would prefer skype for instant messenging, too.

@kduske
Copy link
Collaborator

kduske commented Aug 15, 2012

I have found a bug in the part of the code which I think may be responsible for the crash. It's in the code that generates vector data for the strings to be rendered on screen. I have fixed that bug and uploaded a new version: TrenchBroom_Win32_20120815_2010. If the bug happens again, please post the log file here so that we can start pinpointing it.

@necros0
Copy link
Contributor Author

necros0 commented Aug 15, 2012

I can confirm TrenchBroom_Win32_20120815_2010 does not crash, at least! :)

@kduske
Copy link
Collaborator

kduske commented Aug 18, 2012

So I take it that this one is gone now? hoping

@necros0
Copy link
Contributor Author

necros0 commented Aug 18, 2012

If you are referring to the brush creation crash, then yes, it seems to be fixed. I tested a bit yesterday and made quite a few brushes without any problems. :)

@kduske
Copy link
Collaborator

kduske commented Aug 18, 2012

Good. I'll close this, but if it happens again, be sure to post the log file here.

@kduske kduske closed this as completed Aug 18, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type:Bug Errors and problems
Projects
None yet
Development

No branches or pull requests

2 participants