Adam Heath wrote:
Jacques Le Roux wrote:
Then we should have this kind of advices in the API or the documentation
somewhere. We can't expect that future devs will pick this
message easily. It would be easier for us to refer to somewhere as well.
Create an annotation that marks such things, then use apt to verify it
at the source level.
I plan on going through the project code tonight and change the code
that tests getOriginal() for null. After that I will get rid of the
NullExpr class I just created to fix the problem. Then I will clearly
document the FSE methods and what they return. Maybe include some
example uses in the class description.
You used the term contract in a previous thread. I like that term. The
contract with FSE will be clearly defined when I am finished.
-Adrian
