GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Various Quality of Life features by FransBouma · Pull Request #262 · ReClassNET/ReClass.NET

Conversation

@FransBouma

@FransBouma FransBouma commented Jul 3, 2023 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Hi,

I know, big PRs with multiple features suck, but otherwise I had to create a branch per feature which sucks too. :(

I added several QoL features to the main branch so stuff is now a bit more automated. Main driver is to utilize RTTI information more in an automated fashion. These make ReClass.NET now way more usable with RTTI info containing exe's :)

Stuff changed / added:

  • Defined various keyboard shortcuts for the node types, centrally defined in the Settings class. I also changed the ToolStripButtons to ToolStripMenuItems so they can have shortcuts (they behave the same). This has the advantage that you can just press e.g. Ctrl-Shift-P to define a pointer without opening the context menu as the keyboard shortcut is picked up by the toolbar button :)
  • Moved ReadFromMemory to MemoryBuffer so it's usable from multiple node types
  • VTable node now always displays RTTI info in the comment, if applicable. This information was lost once you created a VTable node
  • Added a way to init a class from RTTI information: it'll name the class after the type associated with the VTable and it will also define the first node as a VTable node.
  • The node context menu is now visible on nested class nodes, and options work now.

I hope I've added all code to the right place. I avoided refactoring whenever possible.

ps: in the commits there are also changes related to alt-shortcuts (showing when pressing alt on main menu items) but these obviously don't show in a context menu so I later on removed them again.

If you want I can also add Undo / Redo to node info. I wrote an algorithm library years ago (https://github.com/SolutionsDesign/Algorithmia) which I use in my software LLBLGen Pro extensively to build undo-redo aware datastructures and it's very easy to add an undo-redo aware property for instance (without the outside code needing any changes).

Hopefully you can use these changes!

(If you think, who are you: I wrote the Universal Unreal Engine Unlocker tools and a lot of other camera mods for games).

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

KN4CK3R, maluramichael, Zyan0x, pinwhell, furyzenblade, Boyarinov, and bukowa reacted with hooray emoji GH-Rake, Zyan0x, pinwhell, Sightem, furyzenblade, Boyarinov, and bukowa reacted with heart emoji
All reactions
  • 🎉 7 reactions
  • ❤️ 7 reactions
FransBouma added 8 commits July 3, 2023 12:23
@FransBouma
Added alt-shortcuts to main context menu
05c13a1
@FransBouma
Defined various shortcut keys on the node type menu items. Changed to…
e3660a7
…olbar button to toolbar menu item so it can have a shortcut too.
@FransBouma
Moved ReadFromBuffer to MemoryBuffer where it belongs, and it's now u…
4d3ac22
…sable from different nodes. Changed VTable node so it always displays the RTTI information if available.
@FransBouma
Redefined some kb shortcuts, as ctrl-shift-c/v aren't used elsewhere
4d2f079
@FransBouma
Added a way to name a class after RTTI information associated with th…
208454e
…e class. This required some hairy work in a lot of places as the context menu wasn't usable on nested classes. It now is
@FransBouma
Changed 'Auto-name' into 'Init from RTTI' and it now also inits the v…
b61edca
…table pointer automatically.
@FransBouma
Removed alt-key shortcuts from context menu as they can't be viewed a…
68b44d5
…nyway
@FransBouma
Added a toolstrip button for Init class from RTTI so the shortcut wor…
c831f9b
…ks without opening the menu. Made the window a bit wider (was too small already) so all toolbar buttons are visible
@FransBouma

FransBouma commented Jul 4, 2023

Copy link
Copy Markdown
Author

Added 2 tiny things:

  • made the initial window a bit wider (was already too narrow) so all toolbar buttons are visible
  • added the init class from rtti to the toolbar so the shortcut ctrl-shift-N is now usable without opening the context menu. Disabled/enabled based on selected node (only enabled on class nodes)
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@FransBouma
Marked Init class from RTTI toolbar button as 'Overflow as needed' so…
dc5225e
… it gets removed when the window is too narrow
KN4CK3R
KN4CK3R reviewed Jul 4, 2023
View reviewed changes
Comment thread ReClass.NET/Controls/MemoryViewControl.cs Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread ReClass.NET/Memory/MemoryBuffer.cs Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread ReClass.NET/Nodes/BaseNode.cs Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread ReClass.NET/Nodes/BaseNode.cs Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread ReClass.NET/Nodes/ClassNode.cs Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread ReClass.NET/Settings.cs Resolved

Uh oh!

There was an error while loading. Please reload this page.

@FransBouma
Removed 2nd empty lines when introduced, added 32bit version of ReadF…
aa8a857
…romBuffer to MemoryBuffer, have refactored Hex32Node to use this method instead, have removed unnecessary braces and brackets.
@FransBouma FransBouma mentioned this pull request Jul 4, 2023
Adds Undo/Redo to ReClass.NET #263
Open
@FransBouma

FransBouma commented Jul 5, 2023

Copy link
Copy Markdown
Author

I've corrected everything, I think. :)

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@dozmus dozmus mentioned this pull request May 8, 2024
Various Quality of Life features dozmus/ReClass.NET#1
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@KN4CK3R KN4CK3R KN4CK3R left review comments

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@FransBouma @KN4CK3R
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.