Mailing List Archive

Ticket #11642: CommandLineParser cannot handle values being passed to boolean options
#11642: CommandLineParser cannot handle values being passed to boolean options
--------------------------------------+-------------------------
Reporter: wagnerrp | Owner: wagnerrp
Type: Bug Report - General | Status: new
Priority: trivial | Milestone: unknown
Component: MythTV - General | Version: Master Head
Severity: low | Keywords:
Ticket locked: 0 |
--------------------------------------+-------------------------
If an option/value pair is processed, split using a equals ('=') sign, and
passed to a boolean option, the parser will attempt to reprocess the two
arguments separately, and get caught in an infinite loop.

--
Ticket URL: <http://code.mythtv.org/trac/ticket/11642>
MythTV <http://www.mythtv.org>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits@mythtv.org
http://www.mythtv.org/mailman/listinfo/mythtv-commits
Re: Ticket #11642: CommandLineParser cannot handle values being passed to boolean options [ In reply to ]
#11642: CommandLineParser cannot handle values being passed to boolean options
----------------------------------+-----------------------------
Reporter: wagnerrp | Owner: wagnerrp
Type: Bug Report - General | Status: new
Priority: trivial | Milestone: unknown
Component: MythTV - General | Version: Master Head
Severity: low | Resolution:
Keywords: | Ticket locked: 0
----------------------------------+-----------------------------

Comment (by Raymond Wagner <rwagner@…>):

In [changeset:"168b8528f39ef8837884f93d8c26f3496868c642/mythtv"]:
{{{
#!CommitTicketReference repository="mythtv"
revision="168b8528f39ef8837884f93d8c26f3496868c642"
Fix infinite reprocessing loop for booleans in command line parser.

This fixes the immediate issue, by allowing the parser loop to
distinguish between the two scenarios, and only attempt reprocessing if
it makes sense to.

Refs #11642

I'm leaving the ticket open, as we should not be reporting an error
prematurely when we're going to attempt reprocessing. We should probably
also accept various forms of "true" and "false" for booleans, rather
than accept no values.
}}}

--
Ticket URL: <http://code.mythtv.org/trac/ticket/11642#comment:1>
MythTV <http://www.mythtv.org>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits@mythtv.org
http://www.mythtv.org/mailman/listinfo/mythtv-commits
Re: Ticket #11642: CommandLineParser cannot handle values being passed to boolean options [ In reply to ]
#11642: CommandLineParser cannot handle values being passed to boolean options
----------------------------------+--------------------------------
Reporter: Raymond Wagner | Owner: Raymond Wagner
Type: Bug Report - General | Status: closed
Priority: trivial | Milestone: 0.27
Component: MythTV - General | Version: Master Head
Severity: low | Resolution: Fixed
Keywords: | Ticket locked: 0
----------------------------------+--------------------------------
Changes (by Stuart Auchterlonie):

* status: new => closed
* resolution: => Fixed
* milestone: unknown => 0.27


Comment:

Closing out old tickets.

--
Ticket URL: <https://code.mythtv.org/trac/ticket/11642#comment:2>
MythTV <http://www.mythtv.org>
MythTV Media Center