Synopsis - Issue Tracker

Bug139 Editing

Title Incorrect error message for --cprocessor
Created on 2007-05-05.12:52:17 by blindvt, last changed 2007-05-12.18:02:28 by admin.
assigned to stefan priority low
type crash severity normal
components platform
status closed resolution fixed
depends on superseded by
Add Comment:
File
CC List:? blindvt, stefan

Messages
Author: blindvt Date: 2007-05-05.12:52:15
Notice the '%s' in this message:
$ synopsis -p C --cprocessor=Comments.SSDFilter...
 Error: %s is not a known comment processor.

Forgot to pass %a to the error() call, like:

Index: synopsis-trunk/scripts/synopsis
===================================================================
--- synopsis-trunk/scripts/synopsis	(revision 1816)
+++ synopsis-trunk/scripts/synopsis	(working copy)
@@ -123,7 +123,8 @@ def make_processor(argv):
         elif o == '--cprocessor':
             if a not in ['previous', 'grouper']:
                 error('%s is not a known comment processor.\n'
-                      '\t possible values are : previous, grouper')
+                      '\t possible values are : previous, grouper'
+		      %a)
             cprocessors.append({'previous':Comments.Previous,
                                 'grouper':Comments.Grouper}[a]())
Author: blindvt Date: 2007-05-05.18:26:59
Updated patch which also does simplify the handling of --cprocessors.

synopsis-0.9.1-bb.opts-cprocessors-simplification.01.diff
Author: admin Date: 2007-05-12.18:02:26
Thanks !
Files
File nameUploaded
synopsis-0.9.1-bb.opts-cprocessors-simplification.01.diff blindvt, 2007-05-05.18:27:00
History
Date User Action Args
2007-05-05 12:52:21blindvtcreate
2007-05-05 18:27:05blindvtsetfiles: + synopsis-0.9.1-bb.opts-cprocessors-simplification.01.diff
messages: + msg683
2007-05-12 18:02:29adminsetstatus: new
nosy: + stefan
messages: + msg690
assignedto: (no value)
bug_type: (no value)
resolution: (no value)