aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #824 from jtattermusch/expose_get_output_fileJan Tattermusch2015-10-144-39/+58
|\ | | | | Expose GetOutputFile in csharp_names.h
| * expose GetOutputFile in csharp_names.hJan Tattermusch2015-09-234-39/+58
| |
* | Merge pull request #875 from tkarls/return_correct_bool_type_from_mapFeng Xiao2015-10-131-1/+1
|\ \ | | | | | | Changed return type from int32 to bool in function returning a bool
| * | Changed return type from int32 to bool in function returning a boolThomas Karlsson2015-10-131-1/+1
| | |
* | | Changed argument typ to uint32 in set function that sets an uint32 valueThomas Karlsson2015-10-131-1/+1
|/ /
* | Merge pull request #815 from TeBoring/third-partyPaul Yang2015-10-0915-348/+37
|\ \ | | | | | | Use std::get where it's available
| * | Down-integrate internal bug fixing changes.Bo Yang2015-10-0415-348/+37
| | |
* | | Use static castHamdi Sahloul2015-10-091-1/+1
| | | | | | | | | Used static cast to improve initalization compatibility with MSVC2010
* | | VS2010 compatibility of address initalizationHamdi Sahloul2015-10-091-1/+1
| | | | | | | | | VS2010 cannot compile json_util_test.cc since it assumes NULL as a zero (int), and therefore cannot convert into to char*
* | | Remove duplicated header fileJisi Liu2015-10-061-218/+0
| | |
* | | fix stringpiece test to use portable hash set.Jisi Liu2015-10-051-1/+1
| | |
* | | Adding back three missing export macros.Jisi Liu2015-10-051-3/+3
| | |
* | | Merge remote-tracking branch 'origin/master' into fix-authorJisi Liu2015-10-0525-35/+282
|\ \ \
| * | | First pass at generating XML documentation from .proto comments.Jon Skeet2015-10-0112-1/+177
| | | | | | | | | | | | | | | | | | | | | | | | This could be tidied up significantly, and at some point we will want to parse the markdown and generate more appropriate XML - but this is definitely better than nothing. Generated code changes coming in next commit.
| * | | Fix typo in oneof case enum commentJon Skeet2015-09-301-1/+1
| | | |
| * | | Move the summary comments above the attributes.Jon Skeet2015-09-292-9/+9
| | | | | | | | | | | | | | | | (Generated code changes coming next...)
| * | | Added documentation to generated code.Jon Skeet2015-09-292-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are now summaries for: - The Types nested class (which holds nested types) - The file descriptor class for each proto - The enum generated for each oneof (Also fixed two typos.) Generated code in next commit.
| * | | Merge pull request #727 from AustinSchuh/arena_bounds_fixFeng Xiao2015-09-241-0/+3
| |\ \ \ | | |_|/ | |/| | Handled blocks being too small in arena.cc
| | * | Handled blocks being too small in arena.ccAustin Schuh2015-08-281-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | When the user passed in a block which was smaller than the Block structure, this code would blow past the end of the memory and crash. Check for that condition.
| * | | Merge pull request #785 from jskeet/csharp-directoriesJan Tattermusch2015-09-236-13/+52
| |\ \ \ | | | | | | | | | | Generate C# directory hierarchy with new option
| | * | | Stop adding a space to the end of lines for descriptor binary data.Jon Skeet2015-09-011-1/+1
| | | | |
| | * | | Specify csharp_namespace in descriptor.protoJon Skeet2015-09-012-4/+3
| | | | |
| | * | | Generate a directory hierarchy based on namespace (C#)Jon Skeet2015-09-012-6/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This introduces a new C# option, base_namespace. If the option is not specified, the behaviour is as before: no directories are generated. If the option *is* specified, all C# namespaces must be relative to the base namespace, and the directories are generated relative to that namespace. Example: - Any.proto declares csharp_namespace = "Google.Protobuf.WellKnownTypes" - We build with --csharp_out=Google.Protobuf --csharp_opt=base_namespace=Google.Protobuf - The Any.cs file is generated in Google.Protobuf/WellKnownTypes (where it currently lives) We need a change to descriptor.proto before this will all work (it wasn't in the right C# namespace) but that needs the other descriptors to be regenerated too. See next commit...
| | * | | Remove vestigial reference to MakeFixedTagJon Skeet2015-09-011-2/+0
| | | | |
| * | | | Add GOOGLE_ATTRIBUTE_NOINLINE to GetArena() and GetMaybeArenaPointer() ↵0xAAE2015-09-151-0/+8
| | |_|/ | |/| | | | | | | | | | methods. This is to avoid "unresolved link" errors in MSVC 2015 during Release build
| * | | Merge pull request #769 from pkasting/metadataFeng Xiao2015-09-101-1/+1
| |\ \ \ | | | | | | | | | | Fix compile failures building protobuf DLLs on MSVC in Chromium's cod…
| | * | | Change to just not marking the class for exportPeter Kasting2015-08-273-45/+3
| | | | |
| | * | | Fix compile failures building protobuf DLLs on MSVC in Chromium's codebase.Peter Kasting2015-08-273-2/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When trying to compile the protobuf code as a DLL, and then compile other DLLs with generated .pb.cc/h files that reference InternalMetadataWithArena::InternalMetadataWithArena(Arena*), MSVC gives an "unresolved external symbol" error. This seems to be due to the function being simultaneously exported and inline. Moving it out-of-line fixes things. There are other functions exported and inline as well but de-inlining them doesn't seem to be necessary to get the build working, and I'd rather de-inline as few functions as possible.
| * | | | Merge pull request #766 from pkasting/hashFeng Xiao2015-09-101-4/+4
| |\ \ \ \ | | | | | | | | | | | | Refine check for unordered_{map|set} availability.
| | * | | | It helps if you write macros correctlyPeter Kasting2015-08-271-2/+2
| | | | | |
| | * | | | Try modifying check in hopes of passing upstream buildPeter Kasting2015-08-271-2/+3
| | | | | |
| | * | | | Review commentsPeter Kasting2015-08-271-2/+1
| | | | | |
| | * | | | Refine check for unordered_{map|set} availability.Peter Kasting2015-08-271-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's not enough to check for C++11 language support, as it's possible for projects to enable C++11 language and library features independently (e.g. Chromium currently does this). Instead, explicitly check the library version to see if it is recent enough to include unordered_{map|set}.
| * | | | | Don't assume char is signed.Brian Silverman2015-09-081-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It isn't always, which causes problems when trying to put negative values into the array with C++11.
| * | | | | Use TEST_TMPDIR for writing temporary files if it's set.Brian Silverman2015-09-081-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | Bazel expects all tests to do this.
| * | | | | Rename 'BYTE_SIZE' macro to 'GOOGLE_PROTOBUF_BYTE_SIZE'kbinani2015-09-041-12/+12
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | Xcode raises warning that says "'BYTE_SIZE' macro redefined". The original 'BYTE_SIZE' macro definition is here, for example: '/Applications/Xcode/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.10.sdk/usr/include/mach/vm_param.h'
* / | | | Down-integrate from google internal.Jisi Liu2015-10-0562-359/+1325
|/ / / /
* | | | Merge remote-tracking branch 'origin/master' into beta-1Feng Xiao2015-08-299-96/+109
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Conflicts: src/google/protobuf/extension_set.h
| * \ \ \ Merge pull request #770 from pkasting/string_spaceFeng Xiao2015-08-284-19/+14
| |\ \ \ \ | | | | | | | | | | | | Move StringSpaceUsedIncludingSelf to lite library.
| | * | | | Move StringSpaceUsedIncludingSelf to lite library.Peter Kasting2015-08-274-19/+14
| | | |/ / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This came up because Chromium downstream modifies the lite library in a way that requires this function, but I'm upstreaming it because based on the comments in repeated_field.h, this ought to allow resolution of an existing hack. I don't know enough about the protobuf code to feel confident trying to resolve this hack myself, so I've merely updated the TODO comments.
| * | | | Merge pull request #771 from pkasting/static_init_1Feng Xiao2015-08-282-63/+78
| |\ \ \ \ | | | | | | | | | | | | Remove a static initializer by removing a global of non-POD type.
| | * | | | Remove a static initializer by removing a global of non-POD type.Peter Kasting2015-08-272-63/+78
| | | |/ / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These are banned by the Google style guide, and Chromium has a hard no-new-static-initializers policy preventing updating to a new version of libprotobuf unless this is resolved. This is the first such change, I'll need to make at least one more in the future. Luckily, the protobuf source tree already has an alternative to static initializers in once.h; use that machinery instead. I defined everything in the .cc file in a blob to replace the old implementation rather than matching the .h layout precisely; let me know if a different ordering is preferred. I also eliminated the macro that used to be used here as spelling everything out only takes one additional line, and the macro didn't actually handle all details of using a particular member variable, just the declaration, so it felt a bit error-prone.
| * | | | Merge pull request #569 from redivo/masterFeng Xiao2015-08-281-5/+5
| |\ \ \ \ | | |_|_|/ | |/| | | Fix GOOGLE_PROTOBUF_ATOMICOPS_ERROR syntax error
| | * | | Fix GOOGLE_PROTOBUF_ATOMICOPS_ERROR syntax errorGeorge Redivo2015-07-061-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's not possible to define "#error" inside a define. It causes 'error: stray ‘#’ in program' compilation error. Now the define GOOGLE_PROTOBUF_ATOMICOPS_ERROR is the error message and it's used along the code together "#error".
| * | | | Merge pull request #768 from pkasting/timeFeng Xiao2015-08-281-1/+0
| |\ \ \ \ | | |_|/ / | |/| | | Delete kNanosPerSecond from time.cc.
| | * | | Delete kNanosPerSecond from time.cc.Peter Kasting2015-08-271-1/+0
| | | |/ | | |/| | | | | | | | | This variable is unused, and thus triggers a build warning on MSVC.
| * | | Merge pull request #746 from zmodem/fix_predict_macros2Feng Xiao2015-08-271-2/+2
| |\ \ \ | | | | | | | | | | Fix the no-op definitions of GOOGLE_PREDICT_{TRUE,FALSE}
| | * | | Fix the no-op definitions of GOOGLE_PREDICT_{TRUE,FALSE}Hans Wennborg2015-08-261-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updating to the current protobuf version caused the following build errors in Chromium when using Clang on Windows: ..\..\third_party\protobuf\src\google/protobuf/stubs/fastmem.h(67,43) : error: equality comparison with extraneous parentheses [-Werror,-Wparentheses-equality] if (GOOGLE_PREDICT_FALSE(n_rounded_down == 0)) { // n <= 7 ~~~~~~~~~~~~~~~^~~~ The problem is that on Windows, GOOGLE_PREDICT_FALSE is #defined to nothing, so the code expands to 'if ((n_rounded_down == 0))', which Clang warns about. Clang would not have warned if the extra parentheses came from the macro, but in this case they don't because the macro is just dropped. Fix this by making the macros behave as an identity function instead of just getting dropped. This is closer to what these macros look like in stubs/port.h internally.
| * | | | Avoid #including system headers from inside a namespace.Peter Kasting2015-08-271-6/+10
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | port.h #includes various headers in order to define byteswap functions, but it currently does so from inside the google::protobuf namespace. This can cause bizarre symbol conflicts and other build errors as these headers' contents are then included inside this namespace. Instead, #include the relevant headers above the namespace declarations.
* | | | Update generated file.Feng Xiao2015-08-261-1/+0
| | | |
generated by cgit v1.2.3 (git 2.39.1) at 2024-06-26 17:47:10 +0000