Beefy Boxes and Bandwidth Generously Provided by pair Networks
XP is just a number
 
PerlMonks  

Re^3: Code cleanup; how best to deal with: defined(%hash) is deprecated at...

by davido (Cardinal)
on May 22, 2014 at 20:49 UTC ( [id://1087136]=note: print w/replies, xml ) Need Help??


in reply to Re^2: Code cleanup; how best to deal with: defined(%hash) is deprecated at...
in thread Code cleanup; how best to deal with: defined(%hash) is deprecated at...

The only final commitment ought to be $ git merge bool_hash. Until then it's pretty easy to git branch -D bool_hash

Even if you've merged, you can still rewind without causing grief, as long as you haven't pushed somewhere where others might suffer. I know not everyone's sold on version control tools, but I find them empowering; I can try things without having to remember every change I made along the way. And if a plan comes together, I can merge. If not, cut off the experimental branch with the confidence of knowing that the stable branch is still untouched.

If your project hasn't been using VC, it's not too late to start. Assuming it's currently in a working state, initialize a repo, create a "refactoring" branch (probably use a more descriptive name), and get to work. At each major decision, branch, then if it works well, merge.


Dave

Replies are listed 'Best First'.
Re^4: Code cleanup; how best to deal with: defined(%hash) is deprecated at...
by taint (Chaplain) on May 23, 2014 at 07:15 UTC
    Hello, davido.

    Good advice. I hope anyone examining this thread, takes your advice to heart. No matter Git, SVN, CVS. Whatever your RCS, VCS. In the end you'll be glad you did -- really. :)

    As for this small endeavor; I already had the whole of the project open in my editor. I think ~90 files. So if anything went south. It was a simple matter of CTRL+Z to back out. :). But as a rule, version tracking/control is my motto.

    In the end, I took Perl's advice (also echoed by others in this thread), and simply removed the defined. That along with your thoughtful "wake-up call" regarding diagnostics, and a few tests, indicated it's all good now -- well that much, anyway.

    Thanks for all your input, davido.
    Much appreciated.

    --Chris

    ¡λɐp ʇɑəɹ⅁ ɐ əʌɐɥ puɐ ʻꜱdləɥ ꜱᴉɥʇ ədoH

Log In?
Username:
Password:

What's my password?
Create A New User
Domain Nodelet?
Node Status?
node history
Node Type: note [id://1087136]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this?Last hourOther CB clients
Other Users?
Others meditating upon the Monastery: (2)
As of 2024-04-20 05:35 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    No recent polls found