-
Notifications
You must be signed in to change notification settings - Fork 857
Merged
Changes from 1 commit
Commits
Show all changes
82 commits
Select commit
Hold shift + click to select a range
3e30055
Made Result contain only an optional pointer to an error
goedderz fd78e02
Fixed compile error
goedderz df2e43c
Avoid no-error allocations
goedderz d447876
WIP
goedderz 4d8d9b1
WIP
goedderz ab77f45
Added concatT and joinT to StringUtils
goedderz 331fad5
Fixed tests
goedderz bfc7901
Merge branch 'feature/string-utils-concat-join' of github.com:arangod…
goedderz 699a5dc
Changed namespace from impl to detail
goedderz 8aa96ae
Assert capacity
goedderz b0e1b0c
Fixed compile errors
goedderz 83bb4f8
Return a string_view in Result::errorMessage
goedderz c2435bc
Removed erroneous line
goedderz 6949f89
Merge branch 'feature/refactor-error-message-string-view' of github.c…
goedderz 0fed08d
Fixed some compile errors
goedderz d91cfb5
Some minor changes
goedderz 6eb6ee1
Merge branch 'feature/refactor-error-message-string-view' of github.c…
goedderz 1956b21
Update lib/Basics/ResultT.h
goedderz f5de874
Merge branch 'feature/refactor-error-message-string-view' of github.c…
goedderz 84511fb
Fixed Mac compile error
goedderz eaacdc0
Merge branch 'feature/string-utils-concat-join' of github.com:arangod…
goedderz d988cf8
Merge branch 'feature/refactor-error-message-string-view' of github.c…
goedderz 6d83aeb
Update lib/Basics/Result.h
goedderz 937877d
Added errorMessage move overloads
goedderz 4d49f20
Added noexcept where mentioned in reviews
goedderz 0e098b2
Be consistent.
goedderz 9c209b3
Merge branch 'devel' of github.com:arangodb/arangodb into feature/sma…
goedderz 9e25112
Removed superfluous lines
goedderz bc3c0ef
Merge branch 'devel' of github.com:arangodb/arangodb into feature/sma…
goedderz 27fb946
Fixed merge conflict
goedderz 0433a27
Minor changes
goedderz 0fa6aa1
Added some assertions
goedderz 76f2312
Pass conflictingDocumentKey separately, no longer in a Result's error…
goedderz 690c2d7
Fixed prior commit
goedderz 01ed70a
Added ErrorCode and changed errorfile generation accordingly
goedderz e57dd14
Regenerate errorfiles if the script changed as well
goedderz 932818c
Make libarango compile with ErrorCode
goedderz 5566c81
Make arango_v8 compile
goedderz fa8baa3
Make arango_network compile
goedderz 7c49d47
Make arango_shell compile
goedderz d381218
Make arango_indexes and arango_cluster_engine compile
goedderz ff8c330
Made arangosh compile
goedderz 4c95166
Made arango_cluster_methods compile
goedderz a52fe06
Made arangobench compile
goedderz b555dc1
Made arangoimport, -backup, -restore, and -dump compile
goedderz b927c56
Made arango_iresearch compile
goedderz dfe105a
Made arango_storage_engine_common compile
goedderz f816181
Fix compile error
goedderz 2fa9254
Made arango_agency compile
goedderz 0fc65b7
Made arango_pregel compile
goedderz a6e7b67
Fixed assertion
goedderz 61e06c6
Made arango_common_rest_handler compile
goedderz 411fec1
Made arango_rocksdb compile
goedderz d2e717f
Made arango_storage_engine compile
goedderz 1a8d2d6
Made arango_replication compile
goedderz c812ab7
Merge branch 'feature/small-result-with-external-error' of github.com…
goedderz 943bf3e
Fix
goedderz e928f04
Only return conflicting key when it's meaningful, to soothe an assertion
goedderz 23b957d
Made arango_vocbase compile
goedderz 0d78b83
Reformat
goedderz ef30942
Added a comment
goedderz a29e7bd
Addressed review comments
goedderz ea52266
Made arango_graph compile
goedderz a4980a4
Made arango_aql compile
goedderz 672b488
Made arango_v8server compile
goedderz 359575c
Made arangosh compile
goedderz aeb1ce9
Made arangoserver compile
goedderz 799d190
Merge commit '0737c305c536043a3cde5916957dd2714836d25f' into tmp
goedderz 2580766
Merge branch 'tmp' into feature/strict-typing-for-error-codes
goedderz 64ffbc6
Merge branch 'devel' of github.com:arangodb/arangodb into feature/str…
goedderz a90d8fb
Made arangodbtests compile
goedderz 92cec6f
Fixed compile errors in CE and tests
goedderz bd5265e
reformat
goedderz 33b212e
Update arangod/GeneralServer/RestHandler.cpp
goedderz 9523d4b
Update lib/Basics/tri-zip.cpp
goedderz eb92d47
Update lib/V8/v8-utils.cpp
goedderz e428231
Addressed review comments
goedderz ba6f370
Merge branch 'feature/strict-typing-for-error-codes' of github.com:ar…
goedderz 15d03dc
Update lib/Basics/files.cpp
goedderz 8fc6866
Merge branch 'devel' of github.com:arangodb/arangodb into feature/str…
goedderz 1815cf1
Fixed merge conflict: regenerated errorfiles
goedderz 3ca1a7b
Merge branch 'devel' of github.com:arangodb/arangodb into feature/str…
goedderz 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
Update lib/V8/v8-utils.cpp
Co-authored-by: Jan <jsteemann@users.noreply.github.com>
- Loading branch information
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Feature/strict typing for error codes #13484
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account