Skip to content

Commit fbb9327

Browse files
committed
Merge pull request square#1080 from condda/master
Naturalize's suggestions
2 parents 8507b5d + b15d70e commit fbb9327

File tree

2 files changed

+19
-19
lines changed

2 files changed

+19
-19
lines changed

picasso/src/main/java/com/squareup/picasso/Request.java

+15-15
Original file line numberDiff line numberDiff line change
@@ -113,45 +113,45 @@ private Request(Uri uri, int resourceId, String stableKey, List<Transformation>
113113
}
114114

115115
@Override public String toString() {
116-
final StringBuilder sb = new StringBuilder("Request{");
116+
final StringBuilder builder = new StringBuilder("Request{");
117117
if (resourceId > 0) {
118-
sb.append(resourceId);
118+
builder.append(resourceId);
119119
} else {
120-
sb.append(uri);
120+
builder.append(uri);
121121
}
122122
if (transformations != null && !transformations.isEmpty()) {
123123
for (Transformation transformation : transformations) {
124-
sb.append(' ').append(transformation.key());
124+
builder.append(' ').append(transformation.key());
125125
}
126126
}
127127
if (stableKey != null) {
128-
sb.append(" stableKey(").append(stableKey).append(')');
128+
builder.append(" stableKey(").append(stableKey).append(')');
129129
}
130130
if (targetWidth > 0) {
131-
sb.append(" resize(").append(targetWidth).append(',').append(targetHeight).append(')');
131+
builder.append(" resize(").append(targetWidth).append(',').append(targetHeight).append(')');
132132
}
133133
if (centerCrop) {
134-
sb.append(" centerCrop");
134+
builder.append(" centerCrop");
135135
}
136136
if (centerInside) {
137-
sb.append(" centerInside");
137+
builder.append(" centerInside");
138138
}
139139
if (rotationDegrees != 0) {
140-
sb.append(" rotation(").append(rotationDegrees);
140+
builder.append(" rotation(").append(rotationDegrees);
141141
if (hasRotationPivot) {
142-
sb.append(" @ ").append(rotationPivotX).append(',').append(rotationPivotY);
142+
builder.append(" @ ").append(rotationPivotX).append(',').append(rotationPivotY);
143143
}
144-
sb.append(')');
144+
builder.append(')');
145145
}
146146
if (purgeable) {
147-
sb.append(" purgeable");
147+
builder.append(" purgeable");
148148
}
149149
if (config != null) {
150-
sb.append(' ').append(config);
150+
builder.append(' ').append(config);
151151
}
152-
sb.append('}');
152+
builder.append('}');
153153

154-
return sb.toString();
154+
return builder.toString();
155155
}
156156

157157
String logId() {

picasso/src/test/java/com/squareup/picasso/UtilsTest.java

+4-4
Original file line numberDiff line numberDiff line change
@@ -108,15 +108,15 @@ public class UtilsTest {
108108

109109
@Test public void getResourceById() throws IOException {
110110
Request request = new Request.Builder(RESOURCE_ID_URI).build();
111-
Resources resources = Utils.getResources(mockPackageResourceContext(), request);
112-
int id = Utils.getResourceId(resources, request);
111+
Resources res = Utils.getResources(mockPackageResourceContext(), request);
112+
int id = Utils.getResourceId(res, request);
113113
assertThat(id).isEqualTo(RESOURCE_ID_1);
114114
}
115115

116116
@Test public void getResourceByTypeAndName() throws IOException {
117117
Request request = new Request.Builder(RESOURCE_TYPE_URI).build();
118-
Resources resources = Utils.getResources(mockPackageResourceContext(), request);
119-
int id = Utils.getResourceId(resources, request);
118+
Resources res = Utils.getResources(mockPackageResourceContext(), request);
119+
int id = Utils.getResourceId(res, request);
120120
assertThat(id).isEqualTo(RESOURCE_ID_1);
121121
}
122122

0 commit comments

Comments
 (0)