1. 24 Sep, 2013 2 commits
  2. 18 Sep, 2013 1 commit
    • Max Cai's avatar
      Add two codegen parameters to nano. · 71766127
      Max Cai authored
      enum_style = c | java: 'c' to put the enum member int constants
      at the parent scope; 'java' to create uninstantiatable shell
      classes at the parent scope and put the int constants inside.
      
      optional_field_style = default | accessors: 'default' to create
      one public mutable field per optional proto field; 'accessors'
      to encapsulate the generated fields behind get, set, has and
      clear accessors.
      
      This CL only contains parsing code for these two parameters.
      
      Change-Id: Iec0c3b0f30af8eb7db328e790664306bc90be089
      71766127
  3. 13 Aug, 2013 1 commit
  4. 08 Aug, 2013 4 commits
  5. 07 Aug, 2013 1 commit
  6. 05 Aug, 2013 1 commit
    • Max Cai's avatar
      Fix outer classname for javamicro/javanano. · 06eed37e
      Max Cai authored
      - File class name is defined as the java_outer_classname option value
        or the file name ToCamelCase; never the single message's ClassName.
      - File-scope enums are translated to constants in the file class,
        regardless of java_multiple_files.
      - If java_multiple_files=true, and file's class name equals a message's
        class name, no error. This is done by detecting that the outer class
        is not needed and skipping the outer class codegen and clash checks.
        Note: there is a disparity between java[lite] and the previous
        java{micr|nan}o: when generating code for a single-message proto, the
        outer class is omitted by java{micr|nan}o if the file does not have
        java_outer_classname. This change makes java{micr|nan}o align with
        java[lite] codegen and create the outer class, but will print some
        info to warn of potential change of code.
      - Also fixed the "is_own_file" detection and made all parseX() methods
        static. Previously, all messages in a java_multiple_files=true file
        are (incorrectly) considered to be in their own files, including
        nested messages, causing them to become inner classes (instance-
        bound) and forcing the parseX() methods to lose the static modifier.
      - This change supersedes c/60164 and c/60086, which causes javanano to
        put enum values into enum shell classes if java_multiple_files=true.
        We now always use the parent class to host the enum values. A future
        change will add a command line option to provide more flexibility.
      - Elaborated in java/README.txt.
      
      Change-Id: I684932f90e0a028ef37c662b221def5ffa202439
      06eed37e
  7. 31 Jul, 2013 3 commits
  8. 29 Jul, 2013 3 commits
  9. 26 Jul, 2013 4 commits
  10. 25 Jul, 2013 2 commits
    • Ulas Kirazci's avatar
      Add test for passing in proto params. · 2000cfdc
      Ulas Kirazci authored
      Depends on Ia532647fe8811d39230a23ba3671685b0388cbe0
      
      Change-Id: I0ba3ce3561aff31e246968a79b1d1053dec91d49
      2000cfdc
    • Max Cai's avatar
      Per-file java_multiple_files flag. · 4c4d6390
      Max Cai authored
      Imported source files may have different values for the 'java_multiple_files'
      option to the main source file's. Whether the fully qualified Java name of an
      entity should include the outer class name depends on the flag value in the
      file defining the referenced entity, not the main file. This CL loads the
      flag values from the main and all transitively imported files into the params,
      and generates the fully qualified Java names accordingly.
      
      If the generator option 'java_multiple_files' is set, its value overrides any
      in-file values in all source/imported files. This is because this generator
      option is typically used on either none or all source files.
      
      Change-Id: Id6a4a42426d68961dc669487d38f35530deb7d8e
      4c4d6390
  11. 23 Jul, 2013 4 commits
  12. 19 Jul, 2013 8 commits
  13. 11 Jul, 2013 3 commits
  14. 09 Jul, 2013 3 commits