Skip to main content

please review doc change

4 replies [Last post]
dlpharness
Offline
Joined: 2007-07-26

Reply viewing options

Select your preferred way to display the comments and click "Save settings" to activate your changes.
bkurotsu
Offline
Joined: 2004-12-13

Still not right when read as a complete paragraph.

This sentence only applies if you turn the option in Preferences on (i.e. is not the default referred to in the previous sentences):

If there is no overlap between the selected tests and the "Tests to run" list in the current configuration, the harness displays an advisory message.

Actually, the sentence above can just go at the end of the paragraph, right after you describe the option.

It doesn't all flow that well, would be best to write the whole paragraph as a easy to understand narrative of the two behaviors. Do you understand it?

dlpharness
Offline
Joined: 2007-07-26

I hope this change reflects the behavior.

https://jtharness.dev.java.net/source/browse/jtharness?view=rev&rev=1631

dawn

bkurotsu
Offline
Joined: 2004-12-13

Integrated. It seems technically correct now. Formatting of the section still seems to be hard to read/understand. I should look at it again using the correct style sheet though...

dlpharness
Offline
Joined: 2007-07-26

Thanks Brian, let me know if you need a change. I marked this CR available.
dawn