Jan, Small changes in grammar and word choice. It's acceptable that English articles are not present in short technical document. Test procedures seem fine.
Here's my patch: 29c29 < Provides test steps for testing liblogsvc library --- > Provides steps for testing liblogsvc library 37c37 < Since at time of being there is no dedicated test driver developed --- > Since at the time being there is no dedicated test driver 44,45c44,45 < [1] Install Target Discovery test driver - it is available < in official IPS repository --- > [1] Install Target Discovery test driver - available > from official IPS repository 84c84 < [4] Test, if setting logging level works --- > [4] Test setting logging level William jan damborsky wrote: > Hi, > > could I please ask for reviewing the fix for following bug ? > > 6591 test procedure for liblogsvc library desired > http://defect.opensolaris.org/bz/show_bug.cgi?id=6591 > > webrev is available at: > http://cr.opensolaris.org/~dambi/bug-6591 > > Thank you, > Jan > > _______________________________________________ > caiman-discuss mailing list > caiman-discuss at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
