- Why do you use a List? Make that a SList. Even better, we know how many subjects there are so we can just use a fixed size array or something.
- Fix the formatting: "null) {", ".append(s".
- A comment before the "return 0" explaining what happened would be nice ("// Events with two subjects with the same URI are not supported."). Also, return a warning so people will notice they're doing it wrong.
- Why do you use a List? Make that a SList. Even better, we know how many subjects there are so we can just use a fixed size array or something.
- Fix the formatting: "null) {", ".append(s".
- A comment before the "return 0" explaining what happened would be nice ("// Events with two subjects with the same URI are not supported."). Also, return a warning so people will notice they're doing it wrong.