Skip to content

Commit

Permalink
GH-39322: [C++] Forward arguments to ExceptionToStatus all the way to…
Browse files Browse the repository at this point in the history
… Status::FromArgs (#39323)

### Rationale for this change

This simplifies the creation of long error messages and leads to the use of a string builder to construct the error message.

### What changes are included in this PR?

 - std::forward in ExceptionToStatus
 - A few nitpicky changes
 - Simplification of the error message text
 - Moving the signature of `CheckIfHierarchicalNamespaceIsEnabled` to `azurefs_internal.h` to reduce the size of `azurefs.h` -- implementation remains in `azurefs.cc`

### Are these changes tested?

Yes. By existing tests.
* Closes: #39322

Authored-by: Felipe Oliveira Carvalho <[email protected]>
Signed-off-by: Felipe Oliveira Carvalho <[email protected]>
  • Loading branch information
felipecrv authored Dec 20, 2023
1 parent 708b973 commit 91b2243
Show file tree
Hide file tree
Showing 4 changed files with 177 additions and 157 deletions.
Loading

0 comments on commit 91b2243

Please sign in to comment.