Commit 267ee9f1 authored by Sudeep Kanav's avatar Sudeep Kanav
Browse files

YELLOW

parent 309b5d79
......@@ -48,17 +48,15 @@ import org.fortiss.pragmatictransformation.service.IPragmaticTransformation;
*
* @author $Author$
* @version $Rev$
* @ConQAT.Rating RED Hash: 96EEE93205D46584612B7B222DA52E5C
* @ConQAT.Rating YELLOW Hash: 62CFB4735D98029F7E5E4B97F8BFD7AD
*/
public class CodeSpecPrepareAddReturnStatement extends CacheSupportedTransformationBase {
/** {@inheritDoc} */
@Override
public Object transform(Object c, IPragmaticTransformation deferredTransformation) {
// TODO(VA) This part with the file project over which you iterate seems very common.
// Could be factorized in a base class. For instance "IterativeTransformationBase".
// (SK) No. I get your point about this the de-multiplexer thing and I want to put more
// thought in it before factorizing it into a base class. Created a task for it 7752.
// TODO 3180 Currently every transformation starts with iterating over the file project.
// This can be and should be made better.
if(c instanceof FileProject) {
getChildrenWithType((FileProject)c, CodeSpecification.class).forEach(
e -> deferredTransformation.buildTransformation(e));
......
......@@ -54,7 +54,7 @@ import org.fortiss.pragmatictransformation.service.IPragmaticTransformation;
*
* @author $Author$
* @version $Rev$
* @ConQAT.Rating RED Hash: E5F4A621B0C586519ED3B8820FBA9A18
* @ConQAT.Rating YELLOW Hash: AE9FB20B5D7C2FF35D5C305A378604DB
*/
public class CodeSpecPrepareToIfElseBlock extends CacheSupportedTransformationBase {
/** {@inheritDoc} */
......@@ -66,7 +66,7 @@ public class CodeSpecPrepareToIfElseBlock extends CacheSupportedTransformationBa
return c;
} else if(c instanceof CodeSpecification) {
// Convert this transformation to iterative transformation.
// TODO 7753 This issue is logged in the ADDS redmine project. It is not public.
// TODO 3179 covert the transformation to iterative transformation
recursivelyApplyTransformation(((CodeSpecification)c).getBody());
// At this point code spec is a nested if else.
}
......
......@@ -40,7 +40,7 @@ import org.fortiss.tooling.kernel.model.INamedElement;
*
* @author $Author$
* @version $Rev$
* @ConQAT.Rating RED Hash: 970C44A4B9F82707F0995405FCFA7532
* @ConQAT.Rating YELLOW Hash: 6A1FAD3BE7518AC66938FE5AADE5B08A
*/
public class NamesTransformation extends CacheSupportedTransformationBase {
......@@ -98,7 +98,7 @@ public class NamesTransformation extends CacheSupportedTransformationBase {
}
// This ridiculous thing is done here because otherwise every time names transformation
// is called id is added to the name.
// TODO 7754 This issue is logged in the ADDS redmine project. It is not public.
// TODO 3178
addToMap(ne, ne);
ne.setName(newName);
}
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment