0007474: B3DPluginEngine stores into a block argument

Viewing Issue Simple Details [ Jump to Notes ]

[ View Advanced ] [ Issue History ] [ Print ]

Category

Severity

Reproducibility

Date Submitted

Last Update

0007474

[Squeak] VM

minor

always

03-08-10 21:54

05-03-10 23:41

Reporter

nicolas cellier

View Status

public

Assigned To

lewis

Priority

normal

Resolution

fixed

Status

resolved

Product Version

trunk

Summary

0007474: B3DPluginEngine stores into a block argument

Description

If Preference for not allowing such assignment is enabled, then VMMaker update won't load.

Most often, these blocks are used as secondparameter of #inject:into: and do not require the assigment.

Additional Information

Problem is located in class method:
translateInDirectory: directory doInlining: inlineFlag
"Special case for the 3D code. Check all the classes' timestamps, not just one"
    | cg fname fstat tStamp|
     fname _ self moduleName, '.c'.
    tStamp _ 0.

    tStamp _ { B3DEnginePlugin. B3DTransformerPlugin. B3DVertexBufferPlugin. B3DShaderPlugin. B3DClipperPlugin. B3DPickerPlugin. B3DRasterizerPlugin} inject: 0 into: [:tS :cl|
        Cannot store into ->tS _ tS max: cl timeStamp].

Should be:
(...snip...) inject: 0 into: [:tS :cl| tS max: cl timeStamp].

Attached Files

Relationships

Notes
(0013528 - 247 - 247 - 247 - 247 - 247 - 247) lewis 03-09-10 01:41	Actually, this plugin is part of package Balloon3D (not VMMaker). The Balloon3D project on SqueakSource already has the fix applied, and this is the version of Balloon3D referenced in the MCM for VMMaker. What repository did your source come from?

(0013747 - 44 - 44 - 44 - 44 - 44 - 44) lewis 05-03-10 23:41	Patch applied in the appropriate respository

Issue History
Date Modified	Username	Field	Change
03-08-10 21:54	nicolas cellier	New Issue
03-08-10 21:54	nicolas cellier	Status	new => assigned
03-08-10 21:54	nicolas cellier	Assigned To	=> lewis
03-09-10 01:41	lewis	Note Added: 0013528
05-03-10 23:41	lewis	Status	assigned => resolved
05-03-10 23:41	lewis	Resolution	open => fixed
05-03-10 23:41	lewis	Note Added: 0013747

Mantis 1.0.8[^]

40 total queries executed.
32 unique queries executed.