updates based on HttpTransport improvements in core lib

[aiuto, 2011-05-03 17:54:45]

http://codereview.appspot.com/4433077/diff/3001/buzz-v1-json-oauth2-cmdline-s...
File
buzz-v1-json-oauth2-cmdline-sample/src/main/java/com/google/api/client/sample/buzz/v1/cmdline/GroupActions.java
(right):

http://codereview.appspot.com/4433077/diff/3001/buzz-v1-json-oauth2-cmdline-s...
buzz-v1-json-oauth2-cmdline-sample/src/main/java/com/google/api/client/sample/buzz/v1/cmdline/GroupActions.java:31:
static void checkToken(Buzz buzz) throws IOException {
This is not really checking a token, that's just a side effect of making a
request that returns very little data. Since we use samples to illustrate what
is going on, that point should be made here.

http://codereview.appspot.com/4433077/diff/3001/buzz-v1-json-oauth2-cmdline-s...
File
buzz-v1-json-oauth2-cmdline-sample/src/main/java/com/google/api/client/sample/buzz/v1/cmdline/OAuth2Native.java
(right):

http://codereview.appspot.com/4433077/diff/3001/buzz-v1-json-oauth2-cmdline-s...
buzz-v1-json-oauth2-cmdline-sample/src/main/java/com/google/api/client/sample/buzz/v1/cmdline/OAuth2Native.java:49:
* NOTE: DES-encrypted file storage is used here as a simple platform-neutral way
to store the
Since we're not doing this until after IO, do you want to revert this, or split
out the other changes?

[yanivi, 2011-05-04 03:49:56]

http://codereview.appspot.com/4433077/diff/3001/buzz-v1-json-oauth2-cmdline-s...
File
buzz-v1-json-oauth2-cmdline-sample/src/main/java/com/google/api/client/sample/buzz/v1/cmdline/GroupActions.java
(right):

http://codereview.appspot.com/4433077/diff/3001/buzz-v1-json-oauth2-cmdline-s...
buzz-v1-json-oauth2-cmdline-sample/src/main/java/com/google/api/client/sample/buzz/v1/cmdline/GroupActions.java:31:
static void checkToken(Buzz buzz) throws IOException {
On 2011/05/03 17:54:45, aiuto wrote:
> This is not really checking a token, that's just a side effect of making a
> request that returns very little data. Since we use samples to illustrate what
> is going on, that point should be made here.

Removed this method.

http://codereview.appspot.com/4433077/diff/3001/buzz-v1-json-oauth2-cmdline-s...
File
buzz-v1-json-oauth2-cmdline-sample/src/main/java/com/google/api/client/sample/buzz/v1/cmdline/OAuth2Native.java
(right):

http://codereview.appspot.com/4433077/diff/3001/buzz-v1-json-oauth2-cmdline-s...
buzz-v1-json-oauth2-cmdline-sample/src/main/java/com/google/api/client/sample/buzz/v1/cmdline/OAuth2Native.java:49:
* NOTE: DES-encrypted file storage is used here as a simple platform-neutral way
to store the
On 2011/05/03 17:54:45, aiuto wrote:
> Since we're not doing this until after IO, do you want to revert this, or
split
> out the other changes?

Reverted for now.  Will revisit this after the version 1.4 release.

[yanivi, 2011-05-04 21:27:05]

Please remember to explicitly LGTM when you feel this is ready to be checked in.

[aiuto, 2011-05-04 22:00:22]

LGTM