All, Apologies on the super late reply. Where did this end up? I don’t see any changes in gerrit (at least in Core) it wasn’t clear to me from the thread whether the change was needed in CDM or in Core.
From: [email protected] [mailto:[email protected]] On Behalf Of seungchul.han Sent: Monday, August 15, 2016 10:16 PM To: 'Brian Hollister' <[email protected]>; [email protected]; [email protected] Subject: Re: [Allseen-core] [Allseen-hae] Use of qcc/SmartPointer.h in Public HAE/CDM Interface Dear all, I agree with Brian Hollister. It'd be better to remove this dependency on qcc before CDM release. So if other members don't oppose to replace qcc/SmartPointer with std::shared_ptr, I will update codes before this Friday. If there is a member to oppose this change? Thanks, Seungchul. From: [email protected]<mailto:[email protected]> [mailto:[email protected]] On Behalf Of Brian Hollister Sent: Monday, August 15, 2016 10:30 AM To: [email protected]<mailto:[email protected]>; [email protected]<mailto:[email protected]> Subject: [Allseen-hae] Use of qcc/SmartPointer.h in Public HAE/CDM Interface I responded to a thread a few weeks ago regarding use of private Core headers in HAE (now CDM). The original consensus was to leave it for 16.04 and address the issue in 16.10. In my message I noted that the use of qcc/SmartPointer.h in the public interface of CDM would require deprecation and would mean we can't outright remove the use of private headers for 16.10. I proposed replacing SmartPointer with std::shared_ptr as the preferred fix. Please refer to the full message in the archives: http://lists.allseenalliance.org/pipermail/allseen-core/2016-July/003222.html http://lists.allseenalliance.org/pipermail/allseen-hae/2016-July/001578.html This issue needs to be addressed prior to the release of CDM 16.04 to avoid carrying this dependency for the foreseeable future. Brian Hollister Software Developer · Two Bulls
_______________________________________________ Allseen-core mailing list [email protected] https://lists.allseenalliance.org/mailman/listinfo/allseen-core
