LGTM! On Feb 7, 2013, at 7:37 AM, Dmitri Gribenko <[email protected]> wrote:
> On Sat, Feb 2, 2013 at 1:14 AM, Dmitri Gribenko <[email protected]> wrote: >> As suggested on IRC, ContainerUSR is not really a CXString, but it >> should own the underlying memory buffer. Thus, it is better to change >> the type to std::string. This will not introduce extra copying >> overhead, since the workaround that is being removed was already >> making a copy. > > Hi Argyrios, > > Can you please review this patch, since Doug is apparently busy? > > Dmitri > > -- > main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if > (j){printf("%d\n",i);}}} /*Dmitri Gribenko <[email protected]>*/ _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
