Shadows-of-Fire / Shadows-of-Greg

An addon for GregTech Community Edition with the goal of making it much more complicated and realistic through the use of GregTech 5 Unofficial and GregTech 6 features.
GNU General Public License v3.0
16 stars 30 forks source link

Adding Multiblock Controllers to the input of the Processing Array crashes worlds #71

Closed ALongStringOfNumbers closed 4 years ago

ALongStringOfNumbers commented 4 years ago

GTCE: 1.10.1.557 SoG: 2.14 Forge: 1.12 - 2847

Adding mutliblock controllers to the input of a Processing Array will crash the world. Some example crashes are:

Electric Blast Furnace Controller: https://pastebin.com/LWYE2iKM

The Processing Array Controller, Assembly Line controller, and others.

This did not occur when a fusion reactor controller was used (For any tier of fusion reactor)

phantamanta44 commented 4 years ago

the check for whether the processing machine is a machine as implemented here is insufficient; the check should ensure that it's not only a machine, but that it has a well-defined voltage so the implicit unboxing operation here doesn't throw an NPE