When loops are nested inside of try catch blocks, the graph parsing algorithm has a hard time figuring out how to order the blocks. Before the patch, ForgeFlower would build the statements from the inside out as BasicBlock -> If -> TryCatch -> Loop, but this patch delays the creation of the trycatch block if there are any unresolved edges in the continueSet. The delay results in the order becoming BasicBlock -> If -> Loop -> TryCatch, which is the correct order. This patch fixes two instances of this bug in RconClient and ChaseServer, eliminating the need for the RconClient patch.
Fixes #73 (ClientThread in mcp is RconClient in moj)
Diff: 1.19.2 diff
When loops are nested inside of try catch blocks, the graph parsing algorithm has a hard time figuring out how to order the blocks. Before the patch, ForgeFlower would build the statements from the inside out as
BasicBlock -> If -> TryCatch -> Loop, but this patch delays the creation of the trycatch block if there are any unresolved edges in the continueSet. The delay results in the order becomingBasicBlock -> If -> Loop -> TryCatch, which is the correct order. This patch fixes two instances of this bug inRconClientandChaseServer, eliminating the need for the RconClient patch.Fixes #73 (
ClientThreadin mcp isRconClientin moj) Diff: 1.19.2 diff