js8544 commented on code in PR #261:
URL: https://github.com/apache/arrow-cookbook/pull/261#discussion_r981124694


##########
cpp/source/basic.rst:
##########
@@ -41,9 +41,9 @@ tedious:
   :caption: Checking the status of every function manually
   :dedent: 2
 
-The macro :c:macro:`ARROW_RETURN_NOT_OK` will take care of some of this
-boilerplate for you.  It will run the contained expression and check the 
resulting
-``Status`` or ``Result`` object.  If it failed then it will return the failure.
+The macros :c:macro:`ARROW_RETURN_NOT_OK` and :c:macro:`ARROW_ASSIGN_OR_RAISE` 
will take care of some of these
+boilerplates for you.  They will run the contained expression and check the 
resulting
+``Status`` or ``Result`` object.  If they failed then they will return the 
failure.

Review Comment:
   Sure. Should I split the section "Working with Status and Result" into two 
or should I keep it but move Result related stuff to a separate paragraph?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to