Skip to content
Merged
Changes from 1 commit
Commits
Show all changes
59 commits
Select commit Hold shift + click to select a range
ee87972
Remove headers from CMake file.
jonsimantov Jun 26, 2023
648ba2d
Find-and-replace change.
jonsimantov Jun 26, 2023
101c476
Format code.
jonsimantov Jun 26, 2023
5bb53a5
Find and replace for std::atomic.
jonsimantov Jun 26, 2023
2ab305f
Format code.
jonsimantov Jun 26, 2023
6f91d88
Update to Dec 2022 release of boringssl.
jonsimantov Jun 26, 2023
058a1c5
Re-add patch file.
jonsimantov Jun 26, 2023
c0e6420
Merge branch 'update-boringssl-xcode' into use_cpp14_memory_classes
jonsimantov Jun 27, 2023
54ae6b5
Switch to std::make_unique.
jonsimantov Jun 27, 2023
a5dae51
Fix make_shared, std::move, and class initialization.
jonsimantov Jun 27, 2023
29e86b9
WIP.
jonsimantov Jun 27, 2023
681fba5
Fix more move syntax.
jonsimantov Jun 27, 2023
bc3d774
Fix more std::move issues.
jonsimantov Jun 27, 2023
8ed952c
Remove debug output
jonsimantov Jun 28, 2023
0ba031b
Merge branch 'update-boringssl-xcode' into use_cpp14_memory_classes
jonsimantov Jun 28, 2023
ea56bb7
Fix Android Firestore test, and Storage assignment.
jonsimantov Jun 28, 2023
df10cf1
Make the callback safer via shared_ptr.
jonsimantov Jun 28, 2023
7da99eb
Merge branch 'main' into use_cpp14_memory_classes
jonsimantov Jun 28, 2023
8af3114
Fix lint errors.
jonsimantov Jun 28, 2023
2dc3263
Merge branch 'main' into use_cpp14_memory_classes
jonsimantov Jun 28, 2023
c49a41d
Format code.
jonsimantov Jun 28, 2023
b2de426
Merge branch 'use_cpp14_memory_classes' of https://github.com/firebas…
jonsimantov Jun 28, 2023
56444e2
More lint fixes.
jonsimantov Jun 28, 2023
c5cedd3
More lint.
jonsimantov Jun 28, 2023
4638b17
Additional lint warning fixes.
jonsimantov Jun 28, 2023
cd9625f
Fix Firestore test.
jonsimantov Jun 28, 2023
caa1c19
Fix more lints.
jonsimantov Jun 28, 2023
8f0ca77
Lint lint lint.
jonsimantov Jun 28, 2023
2e0254a
Merge branch 'main' into use_cpp14_memory_classes
jonsimantov Jun 28, 2023
19ec00f
Change RequestDataPtr to a unique_ptr since STLPort is no longer used.
jonsimantov Jun 28, 2023
66c9d29
Remove Move() and Forward(), replacing with std::move() and std::fowa…
jonsimantov Jun 28, 2023
ec835e1
Merge branch 'use_cpp14_memory_classes' of https://github.com/firebas…
jonsimantov Jun 28, 2023
2740dbf
Format code and lint fixes.
jonsimantov Jun 28, 2023
6aa83b6
Change firebase::Move explicitly to std::move.
jonsimantov Jun 28, 2023
5355b5b
Remove lint warnings for headers.
jonsimantov Jun 28, 2023
352686e
More lint warnings on includes.
jonsimantov Jun 28, 2023
153b6d8
Format code.
jonsimantov Jun 28, 2023
7e52ba5
Return back to shared pointer.
jonsimantov Jun 28, 2023
43d0725
Add note about NewCallback and link to TODO.
jonsimantov Jun 28, 2023
b0db38b
Fix lint error.
jonsimantov Jun 28, 2023
39b6672
Fix a lingering firebase::Move.
jonsimantov Jun 29, 2023
2d1126f
Fix optional_test to mock Move correctly again.
jonsimantov Jun 29, 2023
e0c5f3f
Revert prereqs script, python3 change will be made elsewhere.
jonsimantov Jun 29, 2023
332e417
Lint.
jonsimantov Jun 29, 2023
ebc81d0
Remove references to atomic_test.
jonsimantov Jun 29, 2023
0a5ba19
Fix Move -> std::move once more
jonsimantov Jun 29, 2023
6a7d031
Fix a seg fault race condition on Linux by using a unique_ptr.
jonsimantov Jun 29, 2023
ad52259
Empty commit.
jonsimantov Jun 29, 2023
2359406
Merge branch 'main' into use_cpp14_memory_classes
jonsimantov Jun 29, 2023
3d2b749
Fix test error.
jonsimantov Jun 29, 2023
d2c2298
Fix test compilation error.
jonsimantov Jun 29, 2023
97504c0
Fix additional test build error.
jonsimantov Jun 29, 2023
2c88b27
Vector initializer lists use copy semantics. Switch to push_back sinc…
jonsimantov Jun 29, 2023
8cf9397
Merge branch 'main' into use_cpp14_memory_classes
jonsimantov Jun 30, 2023
85fb83b
Merge branch 'main' into use_cpp14_memory_classes
jonsimantov Jun 30, 2023
de2fee0
Fix Forward --> std::forward from the merged Firestore change.
jonsimantov Jun 30, 2023
94175cf
Merge branch 'main' into use_cpp14_memory_classes
jonsimantov Jul 1, 2023
ccbcbfc
Add callback tests for shared_ptr wrapping a unique_ptr.
jonsimantov Jul 5, 2023
b50cdc2
Merge branch 'main' into use_cpp14_memory_classes
jonsimantov Jul 5, 2023
File filter

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix lint error.
  • Loading branch information
jonsimantov committed Jun 28, 2023
commit b0db38b04567814b99580ffd76a378f6e5c23218
4 changes: 2 additions & 2 deletions firestore/src/android/lambda_event_listener.h
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ namespace firestore {
template <typename T>
class LambdaEventListener : public EventListener<T> {
public:
explicit LambdaEventListener(
LambdaEventListener(
std::function<void(const T&, Error, const std::string&)> callback)
: callback_(std::move(callback)) {
FIREBASE_ASSERT(callback_);
Expand All @@ -52,7 +52,7 @@ class LambdaEventListener : public EventListener<T> {
template <>
class LambdaEventListener<void> : public EventListener<void> {
public:
LambdaEventListener(std::function<void()> callback)
explicit LambdaEventListener(std::function<void()> callback)
: callback_(std::move(callback)) {
FIREBASE_ASSERT(callback_);
}
Expand Down