Beefy Boxes and Bandwidth Generously Provided by pair Networks
Perl Monk, Perl Meditation
 
PerlMonks  

Re^4: poll ideas quest 2019

by Jenda (Abbot)
on Nov 07, 2019 at 14:54 UTC ( #11108444=note: print w/replies, xml ) Need Help??


in reply to Re^3: poll ideas quest 2019
in thread poll ideas quest 2019

You do not need to add several to run into problems if you do not have an agreed upon place to add them.

Are you going to add the single new one at the end? Alphabetically? Under a related module?

Jenda
1984 was supposed to be a warning,
not a manual!

Replies are listed 'Best First'.
Re^5: poll ideas quest 2019
by Eily (Monsignor) on Nov 07, 2019 at 16:25 UTC

    Even if you add it in a random place I don't think adding a single line in a section of ~10 lines (in most cases) makes merging difficult. You might end up with a duplicate if two coders add the same module in two different places. But since auto-merging can lead to other issues*, the proper thing to do is to review the merged code anyway.

    *Possible problems with auto-merging a list of modules include:

    • Two modules that export two functions with the same name
    • The same module included but two coders, but one of them then changing one its options
    • Two different modules included, where one is just a wrapper for the second
    • ...

    I'm not saying that choosing an order for the use statements is a bad idea, though I mostly think it's a question of taste of prettiness of the code (which are important). I'm saying that if your stated reason is to make the list of includes more manageable, either there's something very wrong with your includes, or you have failed to consider solutions that solve more problems.

Log In?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others exploiting the Monastery: (4)
As of 2020-03-29 06:18 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?
    To "Disagree to disagree" means to:









    Results (169 votes). Check out past polls.

    Notices?