Remove svnmerge.py's undocumented long flag name processing behavior, which was replacing embedded "-" characters with "_".  Giovanni Bajo indicates that this was an artifact of some uncommitted code which followed the idiom of the Python optparse module, which actually creates an instance field on the options object (e.g. "opts.flag_name"), as opposed to using a dictionary-like object for the options (the current behavior).
Remove svnmerge.py's undocumented long flag name processing behavior,
which was replacing embedded "-" characters with "_".  Giovanni Bajo
indicates that this was an artifact of some uncommitted code which
followed the idiom of the Python optparse module, which actually
creates an instance field on the options object
(e.g. "opts.flag_name"), as opposed to using a dictionary-like object
for the options (the current behavior).

(Subsequent commits r19072 and r19073 made the same naming convention
change for other elements in the "opts" dictionary.)


* contrib/client-side/svnmerge.py
  Throughout, replace "_" with "-" in the flag names "avail_showwhat",
  "avail_display", "dry_run", "show_changes", "commit_file",
  "record_only".

  (OptBase.__init__): Remove replace of "-" with "_" in long flag names.


Review by: giovannibajo

jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion