problem with SchedXaction's use of kvp_frame keys . . .
Josh Sled
jsled@asynchronous.org
Thu, 23 Aug 2001 22:19:50 -0700
On Fri, Aug 24, 2001 at 01:12:34PM +1000, Robert Graham Merkel wrote:
| Having literal keys with slashes in them is probably not a good idea
| given the function kvp_frame_get_frame_slash.
Yup... I was hoping it would DTRT ... or maybe I just meant to switch it
over to TRT at some point.
Indeed, the intent is to have those keys be in a sub-frame... as they
aren't, and they have /'es in their names, this is bad, ym kay...
I wish there was a kvp_frame_get_slot_slash_path ... especially
since kvp_frame_parse_slash_path is static in kvp_frame.c, making
kvp_frame_get_slot_path_gslist( ..., kvp_frame_parse_slash_path() )
not possible... :(
If this is to be fixed, I'd really like to see it use a
kvp_frame_get_slot_slashpath(...), but the writer always gets to decide. :)
| Where else do I need to go looking to fix this, if I'm reading this right?
dialog-{schedxaction,sxsincelast}.c are the only places I can think of.
...jsled