On Sun, Jan 5, 2014 at 4:49 AM, Hoang-Vu Dang <notifications@github.com>wrote:

> In bindings/java/capstone/Capstone.java:
>
> > @@ -250,6 +257,7 @@ public NativeLong cs_disasm_ex(NativeLong handle, byte[] code, NativeLong code_l
> >    public static final int CS_OPT_SYNTAX_INTEL = 1;  // Intel X86 asm syntax - default syntax on X86 (CS_OPT_SYNTAX,  CS_ARCH_X86)
> >    public static final int CS_OPT_SYNTAX_ATT = 2;    // ATT asm syntax (CS_OPT_SYNTAX, CS_ARCH_X86)
> >    public static final int CS_OPT_ON = 3;  // Turn ON an option - this is default option for CS_OPT_DETAIL
> > +	public static final int CS_OPT_SYNTAX_NOREGNAME = 3; // PPC asm syntax: Prints register name with only number (CS_OPT_SYNTAX)
>
> It must be 3 to work. Is this a mistake to have it been the same value as
> CS_OPT_ON?
>

not really, since they are for different contexts: CS_OPT_ON is for
CS_OPT_DETAIL, and CS_OPT_SYNTAX_NOREGNAME is for CS_OPT_SYNTAX.

these contexts are mutually exclusive so i think that is fine.

thanks.