Prashant Srinivasan wrote: > Hi Siva, > Looks fine. > -ps > > > Sivakumar Shanmugasundaram wrote: > >> Oops. Sorry. Forgot my coffee this morning. >> >> The webrev is at >> http://cr.opensolaris.org/~ss39446/dtrace-ruby2/ >>
Siva, during the ARC case reviews we decided to leave dtrace.h in but not mention it as an public interface correct? So, dtrace.h should be a part of this patch. Thanks, -ps >> Apologies again. >> >> Siva >> >> Sivakumar Shanmugasundaram wrote: >> >> >>> Hmm. Posted this from the Forum itself. I was thinking it will quote the >>> previous replies. >>> >>> http://cr.opensolaris.org/~ss39446/webrev2/webrev/ >>> >>> Apologies for the inconvenience. Please review again. Thanks >>> >>> Regards >>> Siva >>> >>> Sivakumar Shanmugasundaram wrote: >>> >>> >>>> Requesting a review of this webrev again, as I had split the webrevs >>>> to handle the two CRs separately. The new changes for this webrev is >>>> to handle CR 6632022 alone and man page changes. Thanks >>>> >>>> Regards >>>> Siva >>>> This message posted from opensolaris.org >>>> >>>> _______________________________________________ >>>> >>>> >>>> webstack-discuss mailing list >>>> webstack-discuss at opensolaris.org >>>> http://mail.opensolaris.org/mailman/listinfo/webstack-discuss >>>> >>>> >> _______________________________________________ >> >> >> webstack-discuss mailing list >> webstack-discuss at opensolaris.org >> http://mail.opensolaris.org/mailman/listinfo/webstack-discuss >> >> > > _______________________________________________ > > > webstack-discuss mailing list > webstack-discuss at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/webstack-discuss >
