Skip to main content

Please review fixes for 6512858

5 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.
Dawn Phillips

I don't understand. I don't think I deleted anything. I added a column
for precedence to the table, and I reorganized it in the order of
precedence. I added a line about the precedence column.
What am I missing?

dawn

jtharness@mobileandembedded.org wrote:
> Just a few minor comments...
> keywordsTabTE.html - "chose" is spelled wrong, should be choose right?
>
> command/keyword.html - I think the line you deleted is necessary. Even if the operators have been documented, the test suite documentation still defines which words are valid (such as interactive, runtime, etc...).
>
> confEdit/keywords.html - same as the first comment
> [Message sent by forum member 'bkurotsu' (bkurotsu)]
>
> http://forums.java.net/jive/thread.jspa?messageID=269414
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: interest-unsubscribe@jtharness.dev.java.net
> For additional commands, e-mail: interest-help@jtharness.dev.java.net
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: interest-unsubscribe@jtharness.dev.java.net
For additional commands, e-mail: interest-help@jtharness.dev.java.net

bkurotsu
Offline
Joined: 2004-12-13

Oh, you're right, never mind. Just the spelling fixes then?

Dawn Phillips

Fixed in rev 885.

https://jtharness.dev.java.net/source/browse/jtharness/trunk/code/src/co...

Please integrate.

dawn

jtharness@mobileandembedded.org wrote:
> Oh, you're right, never mind. Just the spelling fixes then?
> [Message sent by forum member 'bkurotsu' (bkurotsu)]
>
> http://forums.java.net/jive/thread.jspa?messageID=269588
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: interest-unsubscribe@jtharness.dev.java.net
> For additional commands, e-mail: interest-help@jtharness.dev.java.net
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: interest-unsubscribe@jtharness.dev.java.net
For additional commands, e-mail: interest-help@jtharness.dev.java.net

bkurotsu
Offline
Joined: 2004-12-13

Just a few minor comments...
keywordsTabTE.html - "chose" is spelled wrong, should be choose right?

command/keyword.html - I think the line you deleted is necessary. Even if the operators have been documented, the test suite documentation still defines which words are valid (such as interactive, runtime, etc...).

confEdit/keywords.html - same as the first comment

bkurotsu
Offline
Joined: 2004-12-13

If you fix those things, I'm ok with integration without further review.