| OLD | NEW |
| 1 /* | 1 /* |
| 2 * Copyright 2013 Google Inc. | 2 * Copyright 2013 Google Inc. |
| 3 * | 3 * |
| 4 * Use of this source code is governed by a BSD-style license that can be | 4 * Use of this source code is governed by a BSD-style license that can be |
| 5 * found in the LICENSE file. | 5 * found in the LICENSE file. |
| 6 */ | 6 */ |
| 7 | 7 |
| 8 #include "GrOvalRenderer.h" | 8 #include "GrOvalRenderer.h" |
| 9 | 9 |
| 10 #include "gl/builders/GrGLProgramBuilder.h" | 10 #include "gl/builders/GrGLProgramBuilder.h" |
| (...skipping 76 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 87 | 87 |
| 88 inline bool isStroked() const { return fStroke; } | 88 inline bool isStroked() const { return fStroke; } |
| 89 | 89 |
| 90 class GLProcessor : public GrGLGeometryProcessor { | 90 class GLProcessor : public GrGLGeometryProcessor { |
| 91 public: | 91 public: |
| 92 GLProcessor(const GrBackendProcessorFactory& factory, const GrProcessor&
) | 92 GLProcessor(const GrBackendProcessorFactory& factory, const GrProcessor&
) |
| 93 : INHERITED (factory) {} | 93 : INHERITED (factory) {} |
| 94 | 94 |
| 95 virtual void emitCode(const EmitArgs& args) SK_OVERRIDE { | 95 virtual void emitCode(const EmitArgs& args) SK_OVERRIDE { |
| 96 const CircleEdgeEffect& circleEffect = args.fGP.cast<CircleEdgeEffec
t>(); | 96 const CircleEdgeEffect& circleEffect = args.fGP.cast<CircleEdgeEffec
t>(); |
| 97 GrGLVertToFrag v(kVec4f_GrSLType); | 97 const char *vsName, *fsName; |
| 98 args.fPB->addVarying("CircleEdge", &v); | 98 args.fPB->addVarying(kVec4f_GrSLType, "CircleEdge", &vsName, &fsName
); |
| 99 | 99 |
| 100 GrGLVertexBuilder* vsBuilder = args.fPB->getVertexShaderBuilder();; | 100 GrGLVertexBuilder* vsBuilder = args.fPB->getVertexShaderBuilder();; |
| 101 vsBuilder->codeAppendf("%s = %s;", v.vsOut(), circleEffect.inCircleE
dge().c_str()); | 101 vsBuilder->codeAppendf("\t%s = %s;\n", vsName, circleEffect.inCircle
Edge().c_str()); |
| 102 | 102 |
| 103 GrGLGPFragmentBuilder* fsBuilder = args.fPB->getFragmentShaderBuilde
r(); | 103 GrGLGPFragmentBuilder* fsBuilder = args.fPB->getFragmentShaderBuilde
r(); |
| 104 fsBuilder->codeAppendf("float d = length(%s.xy);", v.fsIn()); | 104 fsBuilder->codeAppendf("\tfloat d = length(%s.xy);\n", fsName); |
| 105 fsBuilder->codeAppendf("float edgeAlpha = clamp(%s.z - d, 0.0, 1.0);
", v.fsIn()); | 105 fsBuilder->codeAppendf("\tfloat edgeAlpha = clamp(%s.z - d, 0.0, 1.0
);\n", fsName); |
| 106 if (circleEffect.isStroked()) { | 106 if (circleEffect.isStroked()) { |
| 107 fsBuilder->codeAppendf("float innerAlpha = clamp(d - %s.w, 0.0,
1.0);", | 107 fsBuilder->codeAppendf("\tfloat innerAlpha = clamp(d - %s.w, 0.0
, 1.0);\n", fsName); |
| 108 v.fsIn()); | 108 fsBuilder->codeAppend("\tedgeAlpha *= innerAlpha;\n"); |
| 109 fsBuilder->codeAppend("edgeAlpha *= innerAlpha;"); | |
| 110 } | 109 } |
| 111 | 110 |
| 112 fsBuilder->codeAppendf("%s = %s;\n", args.fOutput, | 111 fsBuilder->codeAppendf("\t%s = %s;\n", args.fOutput, |
| 113 (GrGLSLExpr4(args.fInput) * GrGLSLExpr1("edge
Alpha")).c_str()); | 112 (GrGLSLExpr4(args.fInput) * GrGLSLExpr1("edge
Alpha")).c_str()); |
| 114 } | 113 } |
| 115 | 114 |
| 116 static void GenKey(const GrProcessor& processor, const GrGLCaps&, | 115 static void GenKey(const GrProcessor& processor, const GrGLCaps&, |
| 117 GrProcessorKeyBuilder* b) { | 116 GrProcessorKeyBuilder* b) { |
| 118 const CircleEdgeEffect& circleEffect = processor.cast<CircleEdgeEffe
ct>(); | 117 const CircleEdgeEffect& circleEffect = processor.cast<CircleEdgeEffe
ct>(); |
| 119 b->add32(circleEffect.isStroked()); | 118 b->add32(circleEffect.isStroked()); |
| 120 } | 119 } |
| 121 | 120 |
| 122 virtual void setData(const GrGLProgramDataManager&, const GrProcessor&)
SK_OVERRIDE {} | 121 virtual void setData(const GrGLProgramDataManager&, const GrProcessor&)
SK_OVERRIDE {} |
| (...skipping 77 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 200 inline bool isStroked() const { return fStroke; } | 199 inline bool isStroked() const { return fStroke; } |
| 201 | 200 |
| 202 class GLProcessor : public GrGLGeometryProcessor { | 201 class GLProcessor : public GrGLGeometryProcessor { |
| 203 public: | 202 public: |
| 204 GLProcessor(const GrBackendProcessorFactory& factory, const GrProcessor&
) | 203 GLProcessor(const GrBackendProcessorFactory& factory, const GrProcessor&
) |
| 205 : INHERITED (factory) {} | 204 : INHERITED (factory) {} |
| 206 | 205 |
| 207 virtual void emitCode(const EmitArgs& args) SK_OVERRIDE { | 206 virtual void emitCode(const EmitArgs& args) SK_OVERRIDE { |
| 208 const EllipseEdgeEffect& ellipseEffect = args.fGP.cast<EllipseEdgeEf
fect>(); | 207 const EllipseEdgeEffect& ellipseEffect = args.fGP.cast<EllipseEdgeEf
fect>(); |
| 209 | 208 |
| 210 GrGLVertToFrag ellipseOffsets(kVec2f_GrSLType); | 209 const char *vsOffsetName, *fsOffsetName; |
| 211 args.fPB->addVarying("EllipseOffsets", &ellipseOffsets); | 210 const char *vsRadiiName, *fsRadiiName; |
| 211 |
| 212 args.fPB->addVarying(kVec2f_GrSLType, "EllipseOffsets", &vsOffsetNam
e, &fsOffsetName); |
| 212 | 213 |
| 213 GrGLVertexBuilder* vsBuilder = args.fPB->getVertexShaderBuilder(); | 214 GrGLVertexBuilder* vsBuilder = args.fPB->getVertexShaderBuilder(); |
| 214 vsBuilder->codeAppendf("%s = %s;", ellipseOffsets.vsOut(), | 215 vsBuilder->codeAppendf("%s = %s;", vsOffsetName, |
| 215 ellipseEffect.inEllipseOffset().c_str()); | 216 ellipseEffect.inEllipseOffset().c_str()); |
| 216 | 217 |
| 217 GrGLVertToFrag ellipseRadii(kVec4f_GrSLType); | 218 args.fPB->addVarying(kVec4f_GrSLType, "EllipseRadii", &vsRadiiName,
&fsRadiiName); |
| 218 args.fPB->addVarying("EllipseRadii", &ellipseRadii); | 219 vsBuilder->codeAppendf("%s = %s;", vsRadiiName, ellipseEffect.inElli
pseRadii().c_str()); |
| 219 vsBuilder->codeAppendf("%s = %s;", ellipseRadii.vsOut(), | |
| 220 ellipseEffect.inEllipseRadii().c_str()); | |
| 221 | 220 |
| 222 // for outer curve | 221 // for outer curve |
| 223 GrGLGPFragmentBuilder* fsBuilder = args.fPB->getFragmentShaderBuilde
r(); | 222 GrGLGPFragmentBuilder* fsBuilder = args.fPB->getFragmentShaderBuilde
r(); |
| 224 fsBuilder->codeAppendf("vec2 scaledOffset = %s*%s.xy;", ellipseOffse
ts.fsIn(), | 223 fsBuilder->codeAppendf("\tvec2 scaledOffset = %s*%s.xy;\n", fsOffset
Name, fsRadiiName); |
| 225 ellipseRadii.fsIn()); | 224 fsBuilder->codeAppend("\tfloat test = dot(scaledOffset, scaledOffset
) - 1.0;\n"); |
| 226 fsBuilder->codeAppend("float test = dot(scaledOffset, scaledOffset)
- 1.0;"); | 225 fsBuilder->codeAppendf("\tvec2 grad = 2.0*scaledOffset*%s.xy;\n", fs
RadiiName); |
| 227 fsBuilder->codeAppendf("vec2 grad = 2.0*scaledOffset*%s.xy;", ellips
eRadii.fsIn()); | 226 fsBuilder->codeAppend("\tfloat grad_dot = dot(grad, grad);\n"); |
| 228 fsBuilder->codeAppend("float grad_dot = dot(grad, grad);"); | |
| 229 | |
| 230 // avoid calling inversesqrt on zero. | 227 // avoid calling inversesqrt on zero. |
| 231 fsBuilder->codeAppend("grad_dot = max(grad_dot, 1.0e-4);"); | 228 fsBuilder->codeAppend("\tgrad_dot = max(grad_dot, 1.0e-4);\n"); |
| 232 fsBuilder->codeAppend("float invlen = inversesqrt(grad_dot);"); | 229 fsBuilder->codeAppend("\tfloat invlen = inversesqrt(grad_dot);\n"); |
| 233 fsBuilder->codeAppend("float edgeAlpha = clamp(0.5-test*invlen, 0.0,
1.0);"); | 230 fsBuilder->codeAppend("\tfloat edgeAlpha = clamp(0.5-test*invlen, 0.
0, 1.0);\n"); |
| 234 | 231 |
| 235 // for inner curve | 232 // for inner curve |
| 236 if (ellipseEffect.isStroked()) { | 233 if (ellipseEffect.isStroked()) { |
| 237 fsBuilder->codeAppendf("scaledOffset = %s*%s.zw;", | 234 fsBuilder->codeAppendf("\tscaledOffset = %s*%s.zw;\n", fsOffsetN
ame, fsRadiiName); |
| 238 ellipseOffsets.fsIn(), ellipseRadii.fsIn(
)); | 235 fsBuilder->codeAppend("\ttest = dot(scaledOffset, scaledOffset)
- 1.0;\n"); |
| 239 fsBuilder->codeAppend("test = dot(scaledOffset, scaledOffset) -
1.0;"); | 236 fsBuilder->codeAppendf("\tgrad = 2.0*scaledOffset*%s.zw;\n", fsR
adiiName); |
| 240 fsBuilder->codeAppendf("grad = 2.0*scaledOffset*%s.zw;", | 237 fsBuilder->codeAppend("\tinvlen = inversesqrt(dot(grad, grad));\
n"); |
| 241 ellipseRadii.fsIn()); | 238 fsBuilder->codeAppend("\tedgeAlpha *= clamp(0.5+test*invlen, 0.0
, 1.0);\n"); |
| 242 fsBuilder->codeAppend("invlen = inversesqrt(dot(grad, grad));"); | |
| 243 fsBuilder->codeAppend("edgeAlpha *= clamp(0.5+test*invlen, 0.0,
1.0);"); | |
| 244 } | 239 } |
| 245 | 240 |
| 246 fsBuilder->codeAppendf("%s = %s;", args.fOutput, | 241 fsBuilder->codeAppendf("\t%s = %s;\n", args.fOutput, |
| 247 (GrGLSLExpr4(args.fInput) * GrGLSLExpr1("edge
Alpha")).c_str()); | 242 (GrGLSLExpr4(args.fInput) * GrGLSLExpr1("edge
Alpha")).c_str()); |
| 248 } | 243 } |
| 249 | 244 |
| 250 static void GenKey(const GrProcessor& processor, const GrGLCaps&, | 245 static void GenKey(const GrProcessor& processor, const GrGLCaps&, |
| 251 GrProcessorKeyBuilder* b) { | 246 GrProcessorKeyBuilder* b) { |
| 252 const EllipseEdgeEffect& ellipseEffect = processor.cast<EllipseEdgeE
ffect>(); | 247 const EllipseEdgeEffect& ellipseEffect = processor.cast<EllipseEdgeE
ffect>(); |
| 253 b->add32(ellipseEffect.isStroked()); | 248 b->add32(ellipseEffect.isStroked()); |
| 254 } | 249 } |
| 255 | 250 |
| 256 virtual void setData(const GrGLProgramDataManager&, const GrProcessor&)
SK_OVERRIDE { | 251 virtual void setData(const GrGLProgramDataManager&, const GrProcessor&)
SK_OVERRIDE { |
| (...skipping 89 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 346 inline Mode getMode() const { return fMode; } | 341 inline Mode getMode() const { return fMode; } |
| 347 | 342 |
| 348 class GLProcessor : public GrGLGeometryProcessor { | 343 class GLProcessor : public GrGLGeometryProcessor { |
| 349 public: | 344 public: |
| 350 GLProcessor(const GrBackendProcessorFactory& factory, const GrProcessor&
) | 345 GLProcessor(const GrBackendProcessorFactory& factory, const GrProcessor&
) |
| 351 : INHERITED (factory) {} | 346 : INHERITED (factory) {} |
| 352 | 347 |
| 353 virtual void emitCode(const EmitArgs& args) SK_OVERRIDE { | 348 virtual void emitCode(const EmitArgs& args) SK_OVERRIDE { |
| 354 const DIEllipseEdgeEffect& ellipseEffect = args.fGP.cast<DIEllipseEd
geEffect>(); | 349 const DIEllipseEdgeEffect& ellipseEffect = args.fGP.cast<DIEllipseEd
geEffect>(); |
| 355 | 350 |
| 356 GrGLVertToFrag offsets0(kVec2f_GrSLType); | 351 const char *vsOffsetName0, *fsOffsetName0; |
| 357 args.fPB->addVarying("EllipseOffsets0", &offsets0); | 352 args.fPB->addVarying(kVec2f_GrSLType, "EllipseOffsets0", |
| 353 &vsOffsetName0, &fsOffsetName0); |
| 358 | 354 |
| 359 GrGLVertexBuilder* vsBuilder = args.fPB->getVertexShaderBuilder(); | 355 GrGLVertexBuilder* vsBuilder = args.fPB->getVertexShaderBuilder(); |
| 360 vsBuilder->codeAppendf("%s = %s;", offsets0.vsOut(), | 356 vsBuilder->codeAppendf("%s = %s;", vsOffsetName0, |
| 361 ellipseEffect.inEllipseOffsets0().c_str()); | 357 ellipseEffect.inEllipseOffsets0().c_str()); |
| 362 | 358 const char *vsOffsetName1, *fsOffsetName1; |
| 363 GrGLVertToFrag offsets1(kVec2f_GrSLType); | 359 args.fPB->addVarying(kVec2f_GrSLType, "EllipseOffsets1", |
| 364 args.fPB->addVarying("EllipseOffsets1", &offsets1); | 360 &vsOffsetName1, &fsOffsetName1); |
| 365 vsBuilder->codeAppendf("%s = %s;", offsets1.vsOut(), | 361 vsBuilder->codeAppendf("\t%s = %s;\n", vsOffsetName1, |
| 366 ellipseEffect.inEllipseOffsets1().c_str()); | 362 ellipseEffect.inEllipseOffsets1().c_str()); |
| 367 | 363 |
| 368 GrGLGPFragmentBuilder* fsBuilder = args.fPB->getFragmentShaderBuilde
r(); | 364 GrGLGPFragmentBuilder* fsBuilder = args.fPB->getFragmentShaderBuilde
r(); |
| 369 SkAssertResult(fsBuilder->enableFeature( | 365 SkAssertResult(fsBuilder->enableFeature( |
| 370 GrGLFragmentShaderBuilder::kStandardDerivatives_GLSLFeature)
); | 366 GrGLFragmentShaderBuilder::kStandardDerivatives_GLSLFeature)
); |
| 371 // for outer curve | 367 // for outer curve |
| 372 fsBuilder->codeAppendf("vec2 scaledOffset = %s.xy;", offsets0.fsIn()
); | 368 fsBuilder->codeAppendf("\tvec2 scaledOffset = %s.xy;\n", fsOffsetNam
e0); |
| 373 fsBuilder->codeAppend("float test = dot(scaledOffset, scaledOffset)
- 1.0;"); | 369 fsBuilder->codeAppend("\tfloat test = dot(scaledOffset, scaledOffset
) - 1.0;\n"); |
| 374 fsBuilder->codeAppendf("vec2 duvdx = dFdx(%s);", offsets0.fsIn()); | 370 fsBuilder->codeAppendf("\tvec2 duvdx = dFdx(%s);\n", fsOffsetName0); |
| 375 fsBuilder->codeAppendf("vec2 duvdy = dFdy(%s);", offsets0.fsIn()); | 371 fsBuilder->codeAppendf("\tvec2 duvdy = dFdy(%s);\n", fsOffsetName0); |
| 376 fsBuilder->codeAppendf("vec2 grad = vec2(2.0*%s.x*duvdx.x + 2.0*%s.y
*duvdx.y," | 372 fsBuilder->codeAppendf("\tvec2 grad = vec2(2.0*%s.x*duvdx.x + 2.0*%s
.y*duvdx.y,\n" |
| 377 " 2.0*%s.x*duvdy.x + 2.0*%s.y
*duvdy.y);", | 373 "\t 2.0*%s.x*duvdy.x + 2.0*%s
.y*duvdy.y);\n", |
| 378 offsets0.fsIn(), offsets0.fsIn(), offsets0.fs
In(), offsets0.fsIn()); | 374 fsOffsetName0, fsOffsetName0, fsOffsetName0,
fsOffsetName0); |
| 379 | 375 |
| 380 fsBuilder->codeAppend("float grad_dot = dot(grad, grad);"); | 376 fsBuilder->codeAppend("\tfloat grad_dot = dot(grad, grad);\n"); |
| 381 // avoid calling inversesqrt on zero. | 377 // avoid calling inversesqrt on zero. |
| 382 fsBuilder->codeAppend("grad_dot = max(grad_dot, 1.0e-4);"); | 378 fsBuilder->codeAppend("\tgrad_dot = max(grad_dot, 1.0e-4);\n"); |
| 383 fsBuilder->codeAppend("float invlen = inversesqrt(grad_dot);"); | 379 fsBuilder->codeAppend("\tfloat invlen = inversesqrt(grad_dot);\n"); |
| 384 if (kHairline == ellipseEffect.getMode()) { | 380 if (kHairline == ellipseEffect.getMode()) { |
| 385 // can probably do this with one step | 381 // can probably do this with one step |
| 386 fsBuilder->codeAppend("float edgeAlpha = clamp(1.0-test*invlen,
0.0, 1.0);"); | 382 fsBuilder->codeAppend("\tfloat edgeAlpha = clamp(1.0-test*invlen
, 0.0, 1.0);\n"); |
| 387 fsBuilder->codeAppend("edgeAlpha *= clamp(1.0+test*invlen, 0.0,
1.0);"); | 383 fsBuilder->codeAppend("\tedgeAlpha *= clamp(1.0+test*invlen, 0.0
, 1.0);\n"); |
| 388 } else { | 384 } else { |
| 389 fsBuilder->codeAppend("float edgeAlpha = clamp(0.5-test*invlen,
0.0, 1.0);"); | 385 fsBuilder->codeAppend("\tfloat edgeAlpha = clamp(0.5-test*invlen
, 0.0, 1.0);\n"); |
| 390 } | 386 } |
| 391 | 387 |
| 392 // for inner curve | 388 // for inner curve |
| 393 if (kStroke == ellipseEffect.getMode()) { | 389 if (kStroke == ellipseEffect.getMode()) { |
| 394 fsBuilder->codeAppendf("scaledOffset = %s.xy;", offsets1.fsIn())
; | 390 fsBuilder->codeAppendf("\tscaledOffset = %s.xy;\n", fsOffsetName
1); |
| 395 fsBuilder->codeAppend("test = dot(scaledOffset, scaledOffset) -
1.0;"); | 391 fsBuilder->codeAppend("\ttest = dot(scaledOffset, scaledOffset)
- 1.0;\n"); |
| 396 fsBuilder->codeAppendf("duvdx = dFdx(%s);", offsets1.fsIn()); | 392 fsBuilder->codeAppendf("\tduvdx = dFdx(%s);\n", fsOffsetName1); |
| 397 fsBuilder->codeAppendf("duvdy = dFdy(%s);", offsets1.fsIn()); | 393 fsBuilder->codeAppendf("\tduvdy = dFdy(%s);\n", fsOffsetName1); |
| 398 fsBuilder->codeAppendf("grad = vec2(2.0*%s.x*duvdx.x + 2.0*%s.y*
duvdx.y," | 394 fsBuilder->codeAppendf("\tgrad = vec2(2.0*%s.x*duvdx.x + 2.0*%s.
y*duvdx.y,\n" |
| 399 " 2.0*%s.x*duvdy.x + 2.0*%s.y*
duvdy.y);", | 395 "\t 2.0*%s.x*duvdy.x + 2.0*%s.
y*duvdy.y);\n", |
| 400 offsets1.fsIn(), offsets1.fsIn(), offsets
1.fsIn(), | 396 fsOffsetName1, fsOffsetName1, fsOffsetNam
e1, fsOffsetName1); |
| 401 offsets1.fsIn()); | 397 fsBuilder->codeAppend("\tinvlen = inversesqrt(dot(grad, grad));\
n"); |
| 402 fsBuilder->codeAppend("invlen = inversesqrt(dot(grad, grad));"); | 398 fsBuilder->codeAppend("\tedgeAlpha *= clamp(0.5+test*invlen, 0.0
, 1.0);\n"); |
| 403 fsBuilder->codeAppend("edgeAlpha *= clamp(0.5+test*invlen, 0.0,
1.0);"); | |
| 404 } | 399 } |
| 405 | 400 |
| 406 fsBuilder->codeAppendf("%s = %s;", args.fOutput, | 401 fsBuilder->codeAppendf("\t%s = %s;\n", args.fOutput, |
| 407 (GrGLSLExpr4(args.fInput) * GrGLSLExpr1("edge
Alpha")).c_str()); | 402 (GrGLSLExpr4(args.fInput) * GrGLSLExpr1("edge
Alpha")).c_str()); |
| 408 } | 403 } |
| 409 | 404 |
| 410 static void GenKey(const GrProcessor& processor, const GrGLCaps&, | 405 static void GenKey(const GrProcessor& processor, const GrGLCaps&, |
| 411 GrProcessorKeyBuilder* b) { | 406 GrProcessorKeyBuilder* b) { |
| 412 const DIEllipseEdgeEffect& ellipseEffect = processor.cast<DIEllipseE
dgeEffect>(); | 407 const DIEllipseEdgeEffect& ellipseEffect = processor.cast<DIEllipseE
dgeEffect>(); |
| 413 | 408 |
| 414 b->add32(ellipseEffect.getMode()); | 409 b->add32(ellipseEffect.getMode()); |
| 415 } | 410 } |
| 416 | 411 |
| (...skipping 822 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 1239 // drop out the middle quad if we're stroked | 1234 // drop out the middle quad if we're stroked |
| 1240 int indexCnt = isStrokeOnly ? SK_ARRAY_COUNT(gRRectIndices) - 6 : | 1235 int indexCnt = isStrokeOnly ? SK_ARRAY_COUNT(gRRectIndices) - 6 : |
| 1241 SK_ARRAY_COUNT(gRRectIndices); | 1236 SK_ARRAY_COUNT(gRRectIndices); |
| 1242 target->setIndexSourceToBuffer(indexBuffer); | 1237 target->setIndexSourceToBuffer(indexBuffer); |
| 1243 target->drawIndexedInstances(kTriangles_GrPrimitiveType, 1, 16, indexCnt
, &bounds); | 1238 target->drawIndexedInstances(kTriangles_GrPrimitiveType, 1, 16, indexCnt
, &bounds); |
| 1244 } | 1239 } |
| 1245 | 1240 |
| 1246 target->resetIndexSource(); | 1241 target->resetIndexSource(); |
| 1247 return true; | 1242 return true; |
| 1248 } | 1243 } |
| OLD | NEW |