Skip to content

libudis86: introduce asmvprintf hook#56

Open
vardyh wants to merge 1 commit intovmt:masterfrom
vardyh:vardyh/devel
Open

libudis86: introduce asmvprintf hook#56
vardyh wants to merge 1 commit intovmt:masterfrom
vardyh:vardyh/devel

Conversation

@vardyh
Copy link

@vardyh vardyh commented Sep 21, 2013

Introduce asmvprintf hook to implement custom assembly code printing
procedure with the help of udis86 builtin syntax formatting machanism.

For syn-intel/att.c code, a syntax element type (enum ud_syn_class)
is passed as the second argument on each calls to asmprintf().

An example of colorized assembly code output is implemented in udcli.c

Signed-off-by: vardyh vardyh.dev@gmail.com

Introduce asmvprintf hook to implement custom assembly code printing
procedure with the help of udis86 builtin syntax formatting machanism.

For syn-intel/att.c code, a syntax element type (enum ud_syn_class)
is passed as the second argument on each calls to asmprintf().

An example of colorized assembly code output is implemented in udcli.c

Signed-off-by: vardyh <vardyh.dev@gmail.com>
@vmt
Copy link
Owner

vmt commented Oct 19, 2013

Hi Vardyh, this is a great change! I apologize for not responding sooner. My only hesitation is with how you disable auto-translate. I think this should be handled properly in the library/api, not something a client does. Let me think about how I can make it easy for you to do this, and merge your change after that.

@vardyh
Copy link
Author

vardyh commented Oct 21, 2013

Hi Vivek, yeah it's true that auto-translate disabling is not implemented in this commit. I did that in the client by setting the translator to NULL, and it's really ugly, and indeed it should not be done like that. Well, let me think about it over and see if there's any better idea of doing this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants