[PATCH] Add ObjCBoxable handling to ObjCMigrator

2015-12-05 Thread AlexDenisov via cfe-commits
data -- AlexDenisov Software Engineer, http://lowlevelbits.org signature.asc Description: Message signed with OpenPGP using GPGMail ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r244193 - [ObjC] Circular containers: add support of subclasses

2015-08-12 Thread AlexDenisov via cfe-commits
Ping -- AlexDenisov Software Engineer, http://lowlevelbits.org > On 06 Aug 2015, at 18:15, Hans Wennborg wrote: > > Hi Alex, > > What crash is that? I don't see any PR number in the commit. > > Richard, what's your owner's opinion? > > On Wed, Aug

Re: r244193 - [ObjC] Circular containers: add support of subclasses

2015-08-06 Thread AlexDenisov via cfe-commits
> Ah, you were referring to > http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20150727/134471.html Yes, that is correct :) > Sorry, email is hard to keep up with sometimes :-) No worries, I do understand ;) -- AlexDenisov Software Engineer, http://lowlevelbits.org > On

Re: r244193 - [ObjC] Circular containers: add support of subclasses

2015-08-05 Thread AlexDenisov
Hi, Hans. The patch fixes crash reported by Argyrios. Might be a candidate for release_37. -- AlexDenisov Software Engineer, http://lowlevelbits.org > On 06 Aug 2015, at 06:51, Alex Denisov <1101.deb...@gmail.com> wrote: > > Author: alexdenisov > Date: Wed Aug 5 23:51:14 2

Re: [PATCH] Fix crash in CheckObjCCircularContainer

2015-08-05 Thread AlexDenisov
s, could you refactor into a function ? Fixed this as well. Also, I got rid of `NSCountedSet` since it’s a subclass of `NSMutableSet` and will be caught by `isSubclassOfNSClass`. The new version of the patch attached. -- AlexDenisov Software Engineer, http://lowl

Re: [PATCH] Fix crash in CheckObjCCircularContainer

2015-08-05 Thread AlexDenisov
Hi guys, any updates? -- AlexDenisov Software Engineer, http://lowlevelbits.org > On 31 Jul 2015, at 22:55, AlexDenisov <1101.deb...@gmail.com> wrote: > >> To clarify, are you saying that the warning may lead to false positives when >> used in subclasses ? > > See