aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* Import iterator_adaptors from google3 (#718)Gravatar Gil2018-01-29
| | | | | | | | | | | | * Import iterator_adapters from google3 * Remove -Wconversion which is annoyingly hard to satisfy * Strip dependency on absl_container from iterator_adapters_test * Format and lint iterator_adaptors * More flexible copyright checking in Travis
* Use fixed-sized types (#719)Gravatar Gil2018-01-28
| | | Should have caught this during review, but cpplint caught it
* Version updates to 4.8.2 (#722)Gravatar Paul Beusterien2018-01-27
|
* Fix a number of c++ build errors (#715)Gravatar rsgowman2018-01-27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Move -fvisibility-inlines-hidden from common_flags to cxx_flags This option isn't supported by C (and causes the build to fail under at least rodete). Manifested error was: ``` -- Looking for include file openssl/rand.h -- Looking for include file openssl/rand.h - not found CMake Error at core/src/firebase/firestore/util/CMakeLists.txt:94 (message): No implementation for SecureRandom available. -- Configuring incomplete, errors occurred! ``` which is completely misleading. :( * Fix exception include for std::logic_error Was causing compiler error on (at least) rodete. (http://en.cppreference.com/w/cpp/error/logic_error suggests that stdexcept is the correct header.) * Remove 'const' from vector contents. vectors cannot contain const objects as the contained objects are required to be assignable and copy constructable. (Not *quite* strictly true; since c++11, this requirement now depends on the operations performed on the container. But my compiler objects at any rate.) http://en.cppreference.com/w/cpp/container/vector https://stackoverflow.com/questions/8685257/why-cant-you-put-a-const-object-into-a-stl-container * Add brackets as suggested (required) by my compiler. * Add missing include For LLONG_MIN, etc. * Enable gnu extension to c++11 for firestore/util *test*. We disable them by default (in cmake/CompilerSetup.cmake) but our tests requires hexidecimal floating point, which is not supported in c++11 (though is supported in C++17, gnu++11, and others). http://en.cppreference.com/w/cpp/language/floating_literal https://bugzilla.redhat.com/show_bug.cgi?id=1321986 * Fix printf format for uint64_t http://en.cppreference.com/w/cpp/types/integer https://stackoverflow.com/questions/9225567/how-to-print-a-int64-t-type-in-c * Allow 0 length printf template strings in tests. * Get rid of a multi line comment. The backslash is apparently interpreted by the compiler cause the next line to be ignored too. In this case, the next line is (currently) a comment, and would be ignored anyways. (But that doesn't stop the compiler from yelling.) * Run ./scripts/style.sh
* Add the C++ linter to the repo (#720)Gravatar Gil2018-01-27
|
* travis: check for copyright in sources (#717)Gravatar Paul Beusterien2018-01-27
|
* Fix b/72502745: OnlineState changes cause limbo document crash. (#470) (#714)Gravatar Michael Lehenbauer2018-01-27
| | | | | | | | | | | | | | | | | | | | [PORT OF https://github.com/firebase/firebase-js-sdk/pull/470] Context: I made a previous change to raise isFromCache=true events when the client goes offline. As part of this change I added an assert for "OnlineState should not affect limbo documents", which it turns out was not valid because: * When we go offline, we set the view to non-current and call View.applyChanges(). * View.applyChanges() calls applyTargetChange() even though there's no target change and it recalculates limbo documents. * When the view is not current, we consider no documents to be in limbo. * Therefore all limbo documents are removed and so applyChanges() ends up returning unexpected LimboDocumentChanges. Fix: I've modified the View logic so that we don't recalculate limbo documents (and generate LimboDocumentChanges) when the View is not current, so now my assert holds and there should be less spurious removal / re-adding of limbo documents.
* normalize and port the rest of Firebase/Port code (#713)Gravatar zxu2018-01-27
| | | | * normalize bits * normalize ordered_code
* port TargetIdGenerator to iOS (#709)Gravatar zxu2018-01-25
| | | | | | | | | | * port TargetIdGenerator to iOS * fix style * move pointer property to instance variable * TriggerTravis
* adding unit test for auto init enable function (#710)Gravatar Chen Liang2018-01-25
|
* implement `TargetIdGenerator` in C++ for Firestore (#701)Gravatar zxu2018-01-25
| | | | | | * implement `TargetIdGenerator` * address changes
* normalize string_util (#708)Gravatar zxu2018-01-25
| | | | * refactoring string_util * port string_util to iOS
* Update CHANGELOG for M21.1 release (#679)Gravatar Ryan Wilson2018-01-25
|
* Fix incorrect deprecation message (#688)Gravatar Iulian Onofrei2018-01-25
| | | | * Fix incorrect deprecation message for Obj-C and Swift
* README - FirebaseCore source pod must be included with all source pods (#705)Gravatar Paul Beusterien2018-01-25
|
* Implement the rest of FieldValue types for C++ (#687)Gravatar zxu2018-01-25
| | | | | | | | | | | | * implement FieldValue for null and boolean. * Implement number and string FieldValue. * Implement object FieldValue. * implement timestamp FieldValue. * Implement number and string FieldValue. * implement public type `Blob` and `GeoPoint` * implement Blob FieldValue * Implement GeoPoint FieldValue * refactoring `Blob`
* Properly publish Abseil sources as a part of the podspec (#704)Gravatar Gil2018-01-24
| | | | | * Properly include abseil sources * Exclude abseil tests
* Adds explicit core graphics dependency to auth (#700)Gravatar Zsika Phillip2018-01-24
|
* Merge pull request #699 from firebase/messaging-copyGravatar Chen Liang2018-01-24
|\ | | | | Adding enable/disable property to FCM token auto Initialization.
| * Merge branch 'master' into messaging-copyGravatar Chen Liang2018-01-24
| |\ | |/ |/|
| * revert unrelated changesGravatar chliangGoogle2018-01-24
| |
| * revert unrelated changesGravatar chliangGoogle2018-01-24
| |
| * revert unrelated changesGravatar chliangGoogle2018-01-24
| |
| * Adding enable/disable property to FCM token auto InitializationGravatar chliangGoogle2018-01-24
| |
* | Update FirebaseMessaging protos (#696)Gravatar Paul Beusterien2018-01-24
| |
* | Disable Messaging build warnings (#697)Gravatar Paul Beusterien2018-01-24
| |
* | Update travis to use CocoaPods 1.4.0 (#692)Gravatar Paul Beusterien2018-01-23
| |
* | Merge pull request #691 from firebase/schmidt-sebastian-patch-5Gravatar Sebastian Schmidt2018-01-22
|\ \ | | | | | | Fixing spelling in FIRStorageErrors
| * | Fixing spelling error in FIRStorageErrorsGravatar Sebastian Schmidt2018-01-22
|/ /
* | Cleanup imports and isEqual (#685)Gravatar Gil2018-01-21
| | | | | | | | | | | | | | | | | | | | * Fix headers * Fix isEqual verbosity * Fix isEqual for nullable properties * Fix nullability on FSTTestDocSnapshot
* | style.sh - output unformatted files in test-only and ignore generated ↵Gravatar Paul Beusterien2018-01-21
| | | | | | | | config.h (#690)
* | Use Comparator in FieldValue. (#686)Gravatar Gil2018-01-19
| |
* | Port comparison to C++ (#678)Gravatar Gil2018-01-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reimplements our comparison functions as C++ Comparators and then provides compatibility shims for interoperating with existing Objective-C usage. A few specialized comparators aren't suitable for porting but only have a single usage (e.g. CompareBytes for comparing NSData * instances). In these cases I've moved them into the caller. * Use int32_t for typeof(ID) in FSTDocumentReference * Migrate callers of FSTComparison.h to Objective-C++ * Port comparison to C++ * Migrate usages of FSTComparison.h to C++ equivalents * Remove FSTComparison
* | Add platform detection logic for SecureRandom (#676)Gravatar Gil2018-01-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add CMake platform detection logic for SecureRandom Now only builds secure_random_arc4random.cc if available. Remove firebase/firestore/base/port.h. Nothing else was in that directory. * Add a SecureRandom implementation that uses OpenSSL This is usable on Linux, Windows, and Android * Properly check return from RAND_bytes
* | Fix build warnings by making void parameter explicit (#681)Gravatar Paul Beusterien2018-01-19
| |
* | Merge pull request #680 from firebase/release-4.8.1Gravatar Paul Beusterien2018-01-18
|\ \ | | | | | | Merge Release 4.8.1
* | | Listen sequence numbers (#675)Gravatar Greg Soltis2018-01-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Generate and save sequence numbers for listens * Add documentation * Fix include path * Fix unavailable comment * Review feedback
* | | Fix CMake build and lint warnings in field_value.cc (#677)Gravatar Gil2018-01-17
| | | | | | | | | | | | | | | | | | | | | Fixes these cpplint warnings: Firestore/core/src/firebase/firestore/model/field_value.cc:162: Semicolon defining empty statement. Use {} instead. [whitespace/semicolon] [5] Firestore/core/src/firebase/firestore/model/field_value.cc:170: Semicolon defining empty statement. Use {} instead. [whitespace/semicolon] [5] Firestore/core/src/firebase/firestore/model/field_value.cc:126: Add #include <utility> for swap [build/include_what_you_use] [4]
* | | Add a cc_library to the CMake build (#670)Gravatar Gil2018-01-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Rewrite cc_test to take named arguments Cut down on build file verbosity by having cc_test take SOURCES and DEPENDS. The separate invocation of target_link_libraries is no longer necessary. * Add a cc_library rule to parallel cc_test This cuts down on build file verbosity. * Automatically add OBJC_FLAGS to cc_libraries if applicable * Exclude platform-specific libraries from 'all' This is makes it possible to declare this kind of library unconditionally. Usage within a test or as a dependency will actually trigger building. * Restore secure_random_test.cc; clean-up comments
* | | Simplify integration with googletest (#672)Gravatar Gil2018-01-17
| | | | | | | | | | | | This makes it possible to build the Firestore subproject with CLion because it no longer needs to be told where googletest is.
* | | implement FieldValue for null, boolean, and array in C++. (#637)Gravatar zxu2018-01-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * implement FieldValue for null and boolean. * refactoring to use union type instead of poly * refactor using union design intead of poly * refactoring to use anonymous union and fix styles * small fix * add field_value_test to the project * fix warning of cmake and fix style
* | | Fix C++ lint errors (#668)Gravatar Gil2018-01-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Misc style.sh fixes * Allow test-only to use a revision; to check your changes since master: ./scripts/style.sh test-only master * Avoid diffing deleted files * 80 columns * Fix C++ lint errors
* | | Enable warnings in the CMake build (#669)Gravatar Gil2018-01-16
| | | | | | | | | | | | | | | | | | * Enable warnings when building with GCC or clang * Fix warnings
* | | Add assert_test to the Xcode build (#671)Gravatar Gil2018-01-16
| | | | | | | | | | | | | | | | | | * Exclude stdio-backed assert from the Xcode build * Add assert_test to the Xcode build
| * | Merge pull request #667 from firebase/bs-cherrypickGravatar Benoit St-Pierre2018-01-15
| |\ \ | | | | | | | | Fix issue @morganchen12 discovered where we weren't properly creating FIRQueryDocumentSnapshot instances. (#662)
* | | | Travis - run tests only for changed code (#665)Gravatar Paul Beusterien2018-01-15
| | | |
| | * | Fix issue @morganchen12 discovered where we weren't properly creating ↵Gravatar Michael Lehenbauer2018-01-15
| |/ / | | | | | | | | | | | | | | | FIRQueryDocumentSnapshot instances. (#662)
* | | Inject infoDictionary to fix flakey tests. (#664)Gravatar Ryan Wilson2018-01-15
| | | | | | | | | | | | | | | | | | * Inject infoDictionary to fix flakey tests. * Remove outdated comment, update format.
* | | Build gRPC for Firestore C++ (#652)Gravatar Gil2018-01-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Clean up quoting and other minor issues * Reorganize CMake build output Make it clearer which parts of the output pertain to external projects. * Use a consistent ordering of ExternalProject arguments * Prevent the top-level build from running in parallel This prevents spurious failures when running make -j. * Actually parse arguments in the xcodebuild function * Use ExternalProject features when available * submodule limits from CMake 3.0 * shallow clones from CMake 3.6 * git progress output from CMake 3.8 * Only build the parts of leveldb we need Skip building the tools and other libraries * Avoid installing ExternalProjects Consume build output directly so that we can build just the targets we need. Installing causes all targets to be built. This doesn't matter as much for these targets but the gRPC build includes a ton of stuff we don't need so it's worth adopting this as a general strategy. * Define an external build for grpc * Test that grpc can link successfully. * Add a FindGRPC CMake module * Actually comment ExternalProjext_GitSource
* | | Validate clang-format compliance in travis (#648)Gravatar Paul Beusterien2018-01-13
| | |
generated by cgit v1.2.3 (git 2.39.1) at 2024-06-20 11:51:57 +0000