Re: svn commit: r1813637 - in /ofbiz/ofbiz-framework/trunk/framework/datafile/src/main/java/org/apache/of biz/datafile: DataFile.java DataFile2EntityXml.java ModelDataFileReader.java Record.java RecordIterator.java

[Michael Brohl]

I think this is just a formatting change becaused I used our formatter for Eclipse.

Not really something worth a discussion, is it?


Am 29.10.17 um 10:07 schrieb Jacques Le Roux:

Le 28/10/2017 à 16:45, mbr...@apache.org a écrit :

-        for (int i = 0; i < modelField.length; i++)
-            sb.append(PAD_CHAR);
-        data = sb.toString();
-        }
+                for (int i = 0; i < modelField.length; i++)
+                    sb.append(PAD_CHAR);
+                data = sb.toString();
+            }

Hi All,

I find this ambiguous. We have few options here:

1)
                for (int i = 0; i < modelField.length; i++)
                    sb.append(PAD_CHAR);

                data = sb.toString();

2)
                for (int i = 0; i < modelField.length; i++) {
                    sb.append(PAD_CHAR);
                }
                data = sb.toString();

3)

                data = sb.toString();

4)


                data = sb.toString();



Thanks

Jacques

                for (int i = 0; i < modelField.length; i++) sb.append(PAD_CHAR);                 for (int i = 0; i < modelField.length; i++) sb.append(PAD_CHAR); Which one do you prefer? Of course this should be a general rule, not only for this case!

smime.p7s
Description: S/MIME Cryptographic Signature