[prev in list] [next in list] [prev in thread] [next in thread] 

List:       xerces-c-dev
Subject:    =?utf-8?q?=5BGitHub=5D_=5Bxerces-c=5D_rleigh-codelibre_commented_on_pull_req?= =?utf-8?q?uest_=2347=
From:       GitBox <git () apache ! org>
Date:       2022-02-02 17:44:06
Message-ID: 164382384688.29368.3097432783595945767.asfpy () gitbox ! apache ! org
[Download RAW message or body]


rleigh-codelibre commented on pull request #47:
URL: https://github.com/apache/xerces-c/pull/47#issuecomment-1028191103


   I am not sufficiently familiar with this part of the codebase to review it \
meaningfully, but the changes look good and the unit tests are passing and not \
reporting any leaks, so I think merging this should be fairly risk-free.  
   Regarding making a new release, all of the recent bugfixes will need backporting \
to the 3.2 branch if we want to have a new 3.2 point release with all of these \
changes included. There are quite a few to backport thanks to all the work done \
recently, primarily by @rouault. @scantor Would you be able to make the release? I \
can probably find some time to do the backporting, unless you want to do this.


-- 
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: c-dev-unsubscribe@xerces.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: c-dev-unsubscribe@xerces.apache.org
For additional commands, e-mail: c-dev-help@xerces.apache.org


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic