[asterisk-bugs] [Asterisk 0013969]: Need tab completion of "core set debug X filename.c"

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13969 
====================================================================== 
Reported By:                jtodd
Assigned To:                mvanbaak
====================================================================== 
Project:                    Asterisk
Issue ID:                   13969
Category:                   General
Reproducibility:            always
Severity:                   trivial
Priority:                   low
Status:                     assigned
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 159221 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-11-25 13:00 CST
Last Modified:              2008-12-04 15:49 CST
====================================================================== 
Summary:                    Need tab completion of "core set debug X filename.c"
Description: 

There is a method by which debugging can be activated for particular C
files by typing something like this from the CLI:

core set debug 2 chan_sip.c

There is no indication that this is possible in the tab completion of the
command line.  After typing "2" in the example above, hitting "tab" gives a
beep instead of something like "<filename.c>" or a list of available files.
 For that matter, the idea that a number can be put in as a debug level is
not clear, either.
====================================================================== 

---------------------------------------------------------------------- 
 (0095805) Corydon76 (administrator) - 2008-12-04 15:49
 http://bugs.digium.com/view.php?id=13969#c95805 
---------------------------------------------------------------------- 
[15:26:03] <Corydon76-dig> russellb: would you agree that tab-completing a
subset of available options is worse than not tab-completing at all?
[15:26:18] <russellb> good question
[15:26:53] <russellb> I suppose so
[15:27:01] <Corydon76-dig> M13969
[15:27:03] <russellb> I'm just concerned that people use tab to see what's
available, and if it shows something, but not everything ...
[15:27:03] <MuffinMan> [assigned] [Asterisk] General 0013969: Need tab
completion of "core set debug X filename.c" reported by jtodd (Karma:
+26.75) http://bugs.digium.com/view.php?id=13969
[15:27:12] <Corydon76-dig> I've been trying to convince jtodd of that
[15:27:32] <russellb> what can't be tab completed in this case?
[15:27:37] <russellb> filenames?
[15:27:40] <eliel> numbers
[15:27:48] <Corydon76-dig> All filenames available for 'core set debug'
[15:27:50] <eliel> verbose levels
[15:27:51] <eliel> and filenames
[15:28:17] <Corydon76-dig> because some of the filenames are not directly
available in the binary, but only by searching the source directory
[15:28:19] <eliel> the problem is also when the next parameter is a
number
[15:28:40] <Corydon76-dig> Right, because the number can be anywhere from
0 to 2.1 billion
[15:28:51] <Corydon76-dig> (all of which are reasonable)
[15:28:59] <russellb> ok, so clearly the number isn't going to fly
[15:29:04] <russellb> filenames we might be able to do
[15:29:12] <russellb> since we already have the
ASTERISK_REGISTER_FILE_VERSION macro
[15:29:17] <russellb> it's registering filenames into a list
[15:29:26] <russellb> so you can pull from that
[15:29:33] <Corydon76-dig> That's a good hint.
[15:29:37] <russellb> :)
[15:29:55] <eliel> wow... i miss that one
[15:30:12] <russellb> check out "core show file version"
[15:30:38] <eliel> exactly what we need
[15:30:42] <russellb> yay
[15:31:18] <russellb> it just means we have to make sure every file has
that macro in it ...
[15:31:27] <russellb> which I'm sure a number don't
[15:32:13] <Corydon76-dig> but some of those files don't have ast_debug()
in them, so it probably doesn't matter
[15:32:20] <russellb> true
[15:32:24] <russellb> (or ast_verb())
[15:32:53] <Corydon76-dig> You can set verbose only for a single file,
too?
[15:33:03] <russellb> in trunk/1.6, yeah 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-12-04 15:49 Corydon76      Note Added: 0095805                          
======================================================================