From ebae9a933b3a82c82b65cbf5bb1bed20f3b934a2 Mon Sep 17 00:00:00 2001 From: Kurt Date: Wed, 14 Oct 2015 20:10:09 -0700 Subject: [PATCH] Cleanup of Delete/View Removed unneeded picturebox array (duplicate of slotpictureboxes) Removed temporary variable 'picturebox', just straight up compare to null. Deleting now instantly sets the picture to null, instead of generating the image. (might not have been the best idea if I somehow decided to change slot filling) Empty slots now instantly return null (empty) for their image instead of doing layering. --- PKX/f1-Main.cs | 23 ++++------------------- 1 file changed, 4 insertions(+), 19 deletions(-) diff --git a/PKX/f1-Main.cs b/PKX/f1-Main.cs index ce116bada..26c95dbfb 100644 --- a/PKX/f1-Main.cs +++ b/PKX/f1-Main.cs @@ -2552,21 +2552,7 @@ private void clickView(object sender, EventArgs e) int slot = getSlot(sender); int offset = getPKXOffset(slot); - PictureBox[] pba = { - bpkx1, bpkx2, bpkx3, bpkx4, bpkx5, bpkx6, - bpkx7, bpkx8, bpkx9, bpkx10,bpkx11,bpkx12, - bpkx13,bpkx14,bpkx15,bpkx16,bpkx17,bpkx18, - bpkx19,bpkx20,bpkx21,bpkx22,bpkx23,bpkx24, - bpkx25,bpkx26,bpkx27,bpkx28,bpkx29,bpkx30, - - ppkx1, ppkx2, ppkx3, ppkx4, ppkx5, ppkx6, - bbpkx1,bbpkx2,bbpkx3,bbpkx4,bbpkx5,bbpkx6, - - dcpkx1, dcpkx2, gtspkx, fusedpkx,subepkx1,subepkx2,subepkx3, - }; - - PictureBox picturebox = pba[slot]; - if (picturebox.Image == null) + if (SlotPictureBoxes[slot].Image == null) { System.Media.SystemSounds.Exclamation.Play(); return; } // Load the PKX file @@ -2651,8 +2637,7 @@ private void clickDelete(object sender, EventArgs e) if (slot == 30 && setParty() == 1 && !DEV_Ability.Enabled) { Util.Alert("Can't delete first slot."); return; } int offset = getPKXOffset(slot); - byte[] pkxdata = new byte[0x104]; - byte[] ekxdata = PKX.encryptArray(pkxdata); + byte[] ekxdata = PKX.encryptArray(new byte[0x104]); savedited = true; @@ -2662,7 +2647,7 @@ private void clickDelete(object sender, EventArgs e) { Array.Copy(ekxdata, 0, savefile, offset, 0xE8); } else return; - getQuickFiller(SlotPictureBoxes[slot], pkxdata); + SlotPictureBoxes[slot].Image = null; getSlotColor(slot, Properties.Resources.slotDel); } private void clickClone(object sender, EventArgs e) @@ -2945,7 +2930,7 @@ private void getSlotFiller(int offset, PictureBox pb) return; } pb.BackColor = Color.Transparent; - pb.Image = p.Sprite; + pb.Image = p.Species == 0 ? null : p.Sprite; } private void getSlotColor(int slot, Image color) {