And again, here we go:

http://cr.openjdk.java.net/~erikj/8023216/webrev.root.01/

/Erik

On 2013-08-19 11:05, Erik Joelsson wrote:
Thanks for the feedback!

I took most of it and made into a change. Also removed some obsolete parts that I stumbled over.

Warn is supposed to be quiet, but at least hotspot is still being quite noisy.

/Erik

On 2013-08-06 01:18, Dan Smith wrote:
I'm building for the first time under the new infrastructure (late to the party, I know). I appreciate the comprehensive readme page. Here's a list of minor points of feedback:

"The set of repositories and what they contain": need to add nashorn

"warn — Default and very quiet": I presume this is still an in-progress goal? I'm seeing almost 3000 lines of text in a fresh build (including over 1000 from 'images')

---

Some copy editing:

"each repository is it's own independent repository": it's ->  its

"Insure that GNU make": insure ->  ensure

"rather thane excludes": thane ->  than

"Other depends on pieces of code": other depends ->  others depend

"for all classes that contains constants": contains ->  contain

"we have missed to add support": missed ->  neglected

Reply via email to