diff options
author | Jim Shuma <jshuma@google.com> | 2010-07-07 14:24:21 -0700 |
---|---|---|
committer | Jim Shuma <jshuma@google.com> | 2010-07-15 13:42:02 -0700 |
commit | 288c8711a64893acb3f4a31caf69153be9809d17 (patch) | |
tree | 4349a0c89d71bfcdd08b70590cd864325ee2f9f5 /graphics/java/android/renderscript/ProgramFragment.java | |
parent | d0d5c072aa8aa30cde3213af8b022104aabb51a5 (diff) |
Making Program*.Builder classes' setters return 'this'
This is for better correspondence with the Builder pattern.
See also: Effective Java, 2nd edition.
Change-Id: Iecccd42be49dea2ed8e4b8cc38ce96379cf3c11c
Diffstat (limited to 'graphics/java/android/renderscript/ProgramFragment.java')
-rw-r--r-- | graphics/java/android/renderscript/ProgramFragment.java | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/graphics/java/android/renderscript/ProgramFragment.java b/graphics/java/android/renderscript/ProgramFragment.java index 5e04f0c57526..d06d768ef423 100644 --- a/graphics/java/android/renderscript/ProgramFragment.java +++ b/graphics/java/android/renderscript/ProgramFragment.java @@ -106,16 +106,18 @@ public class ProgramFragment extends Program { mPointSpriteEnable = false; } - public void setTexture(EnvMode env, Format fmt, int slot) + public Builder setTexture(EnvMode env, Format fmt, int slot) throws IllegalArgumentException { if((slot < 0) || (slot >= MAX_TEXTURE)) { throw new IllegalArgumentException("MAX_TEXTURE exceeded."); } mSlots[slot] = new Slot(env, fmt); + return this; } - public void setPointSpriteTexCoordinateReplacement(boolean enable) { + public Builder setPointSpriteTexCoordinateReplacement(boolean enable) { mPointSpriteEnable = enable; + return this; } public ProgramFragment create() { |