Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 332657 - Troubles with Indigo M4 acceleo update site : NPE visitAcceleoForBlock(AcceleoEvaluationVisitor.java:384)
Summary: Troubles with Indigo M4 acceleo update site : NPE visitAcceleoForBlock(Accele...
Status: CLOSED FIXED
Alias: None
Product: Acceleo
Classification: Modeling
Component: Core (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-12-15 11:42 EST by Fabien Giquel CLA
Modified: 2010-12-16 06:47 EST (History)
3 users (show)

See Also:

Attachments
stack trace (2.79 KB, text/plain)
2010-12-15 11:45 EST, Fabien Giquel CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Fabien Giquel CLA 2010-12-15 11:42:04 EST 
Hi,

on Modisco project, when taking in account last Acceleo update site for M4 build (3.1.0.v20101214-0511), we have some troubles with all our generation junit tests :

NPE at org.eclipse.acceleo.engine.internal.evaluation.AcceleoEvaluationVisitor.visitAcceleoForBlock(AcceleoEvaluationVisitor.java:384)

It seems that this method has changed (calling now getEvaluationVisitor() which returns null because the visitor field points to a AcceleoEvaluationVisitor and not a AcceleoEvaluationVisitorDecorator).
Comment 1 Fabien Giquel CLA 2010-12-15 11:45:21 EST 
Created attachment 185243 [details]
stack trace
Comment 2 Laurent Goubet CLA 2010-12-15 12:51:29 EST 
Hi Fabien,

This is a bug I have already fixed on HEAD ... and can't believe I let through for the actual build.

There is no workaround for this issue, and it will pop up for any and all "for" blocks (most of our own unit cases were failing ... yet they haven't been launched by hudson for this build). We've already settled for an M4b build, it should be available on the Acceleo update site starting tomorrow.

Leaving the bug open till the build is out and available.
Comment 3 Laurent Goubet CLA 2010-12-16 03:55:47 EST 
Hi Fabien,

The 3.1.0M4b build is out. Can you confirm that it fixes this on your side?
Comment 4 Fabien Giquel CLA 2010-12-16 05:14:52 EST 
Hi Laurent, I am looking at the update. 
It works really better now. However i have a small difference in generated output, but it may be caused by my recent templates modification tentatives. I confirm you definitively as soon as possible.
Comment 5 Fabien Giquel CLA 2010-12-16 06:47:01 EST 
Yes, I confirm that the 3.1.0M4b update site fixed the problem.