Fixed Issue #205

[Astabol]

Hello @qwsdl , This is the fixed for Fixed Issue #205.

Thank You

[deRemo]

Thanks @Astabol, please remove the comments you have introduced, since your changes are self-explanatory

[Astabol]

Sure. I shall remove. Thanks

[Astabol]

Hello @deRemo, I removed the comments. Thank You

[deRemo]

Thanks @Astabol, but please squash the commits into one for a cleaner git history

I generally use git rebase -i HEAD~X where X represents the number of commits you want to go back in your history to re-write. Then fix-up the latest commits to the previous one

More info: https://garrytrinder.github.io/2020/03/squashing-commits-using-interactive-rebase

[Astabol]

Hi @deRemo , I squashed the commits to single. Thank You

[deRemo]

Is this required for issue #205? I'd rather not touch the base simulation engine: I'm pretty sure that we want the simulator to fail if getIncomingEvents() is null-pointer, rather than "hiding" it

[Astabol]

Sure. I understand.

[deRemo]

As my comment below, I'm not sure that hiding the nullpointer is the right approach. When calling getCurrentRequestedMips we basically assume that the Vm has been placed, hence getHost() shouldn't be null. I believe that the issue to be fixed is more intricate than this...

[Astabol]

Yeah, it is not fixing the root cause. If anyone called this from another example might get the same issue. I will look into it. Thank You

[deRemo]

Thanks for your time!