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

RAM watch : A "Size" columns #683

Closed
Niamek opened this issue Aug 16, 2016 · 4 comments

Comments

Projects
None yet
4 participants
@Niamek
Copy link

commented Aug 16, 2016

Currently there is only one way(to my knowledge) to check the size of an address: Editing the address. It would be neat if the size was already displayed in a column like the Value, Address,Notes, Prev or the Changes columns.

@adelikat

This comment has been minimized.

Copy link
Contributor

commented Nov 9, 2016

There's other aspects to an address too, such as whether it is big endian, and its display type. Shouldn't those be columns too then? That's an awful lot of columns though

@Niamek

This comment has been minimized.

Copy link
Author

commented Nov 10, 2016

I don't know about these.

The reason why I was interested in the byte size is because when I was writting lua scripts, I had to check the size for every single address I found. Just having a way to display them would be nice. If these information you mentionned, like big endian, could be needed when writting a script, then I'd suggest making an option for them too.

I'd also suggest that they are always set them off by default.

@zeromus zeromus added the App-EmuHawk label Jul 10, 2017

@YoshiRulz

This comment has been minimized.

Copy link
Member

commented Oct 22, 2017

Don't know why display type needs to be a column when there are only a handful. Given the limited number of options for each of size, type and endianness, using one character for each more than satisfies the use case but doesn't hog space. Examples: "4hB" for 4 byte, hex, BE; "1uL" for 1 byte, unsigned, LE

@adelikat

This comment has been minimized.

Copy link
Contributor

commented Nov 1, 2017

I like this suggestion :)

alyosha-tas added a commit that referenced this issue Nov 14, 2017

Merge pull request #1052 from YoshiRulz/ramwatch-displaycol
Add display type column (resolves #683)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.