Description Sebastian Zarnekow 2010-08-06 08:59:47 EDT

The NodeContentAdapter is responsible for syncing the offset information in the node model when it's changed. This is implemented by means of an EContentAdapter although the only interesting notifier type is the CompositeNode. Furthermore, nobody should alter the node model anyway besides the partial parser - which we have under full control.

To get rid of the more or less expensive addNodeContentAdapter(rootNode) we should find another way to sync the offset information, e.g. by explicitly triggering a method like NodeContentAdapter#updateCompositeNode from the parser.

See also StackOverflowError when adding the NodeContentAdapter to a deep expression tree: https://build.eclipse.org/hudson/job/Xtext-nightly-HEAD/94/consoleText