Skip to content

Commit 041343e

Browse files
committed
allow setting required fields with set(Int, Any) on builder
Signed-off-by: Jeff Plaisance <jeffplaisance@gmail.com>
1 parent 02747ba commit 041343e

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

lib/scala-protobuf_2.8.0-0.1.jar

13 Bytes
Binary file not shown.

src/main/scala/protobuf/compiler/ScalaProtoWrapperGenerator.scala

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -175,9 +175,9 @@ object ScalaProtoWrapperGenerator {
175175
}
176176
out.println
177177
out.println(" def set(i:Int, fieldValue:Option[Any]):Unit = {")
178-
if (!optionalFields.isEmpty) {
178+
if (!requiredFields.isEmpty || !optionalFields.isEmpty || !repeatedFields.isEmpty) {
179179
out.println(" i match {")
180-
optionalFields.foreach(field => out.println(" case "+field.getNumber+" => "+field.getName+" = fieldValue.asInstanceOf[Option["+getTypeString(field, javaClass)._1+"]]"))
180+
(requiredFields++optionalFields).foreach(field => out.println(" case "+field.getNumber+" => "+field.getName+" = fieldValue.asInstanceOf[Option["+getTypeString(field, javaClass)._1+"]]"))
181181
out.println(" }")
182182
}
183183
out.println(" }")

0 commit comments

Comments
 (0)