mohitgargk/jCAE — comment on commit 7198641e0f9ac1f630c9541774eff421bdad026e
File:  line 0
By mohitgargk:

- If someone create a file in the mymesh.amibe directory it will through a UnsupportedOperationException
Yes, I am aware of this. I left it as it is because it was not pertinent from the fileChangeEvent demonstration point of view. I will fix this.

- ViewGroupAction should not expose the FileChangeListener publically. The FileChangeListener should be an anonymous class or a private class of ViewGroupAction.
Right, I will modify accordingly.

- Remove noise from your patch like line ViewGroupAction:127 or 105, AbstractGroupAction.java:85 etc...
Ok, This must be because of editor. Instead of cleaning my jCAE copy and putting in github, I checkout the code and did modifications in the 2 files using gedit. This must have introduced these wild characters.

- just a little optimization: instead of monitoring the whole mymesh.amibe directory just monitor mymesh.amibe/jcae3d.
I did that only in the first place. But to keep demonstration purpose simpler I removed that code.

- Fix your tabulations (use tab not 8 spaces)
I will take care of this.

- There can be more than one mesh visible so you cannot have a AbstractGroupAction.temp attribute.
- Do not refresh all visible mesh, refresh only the one which as changed.
I intended to store the node from where the Remesh action originated. I will see for an alternate how to do this.