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