Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Review: import a few Java files #2

Closed
GoogleCodeExporter opened this issue Apr 20, 2015 · 4 comments
Closed

Review: import a few Java files #2

GoogleCodeExporter opened this issue Apr 20, 2015 · 4 comments

Comments

@GoogleCodeExporter
Copy link

Initial import of a few Java files, asking for a code review.


Original issue reported on code.google.com by feng.qia...@gmail.com on 25 Aug 2006 at 11:02

Attachments:

@GoogleCodeExporter
Copy link
Author

You should add a license block to the top of each file.  Other than that, looks 
good.
 r=me.

Original comment by mmento...@gmail.com on 26 Aug 2006 at 2:43

  • Changed title: Review: import a few Java files

@GoogleCodeExporter
Copy link
Author

Sorry, I send out files in a wrong directory. I made copyright changes in 
another
working directory already. Now I re-attach the diff here. In this diff, I added 
all
Java files under java/common/src/com/google/airbag/common, and a build.xml 
script.

The code doesn't compile yet, because it depends on servlet api. A third-party
library will be checked in later.

I am curious what tools you used to do review. Is it hard?

Thanks.

Original comment by feng.q...@gmail.com on 28 Aug 2006 at 5:49

Attachments:

@GoogleCodeExporter
Copy link
Author

Original comment by mmento...@gmail.com on 29 Aug 2006 at 7:25

@GoogleCodeExporter
Copy link
Author

Original comment by mmento...@gmail.com on 29 Aug 2006 at 8:45

  • Changed state: Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant