From: SourceForge.net <no...@so...> - 2006-09-12 09:09:57
|
Patches item #1552007, was opened at 2006-09-04 17:12 Message generated for change (Comment added) made by ikonst You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=381349&aid=1552007&group_id=24366 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: keyboard Group: None Status: Open Resolution: None Priority: 5 Submitted By: Nobody/Anonymous (nobody) Assigned to: Peter Åstrand (astrand) Summary: change the handling of KEYMAP_PATH Initial Comment: Currently (1.4.1), KEYMAP_PATH and mapname are simply merged (see xkeymap.c:xkeymap_read()). This may lead to problems, if you set a different path while passing (for what reason ever) another path to make and you forgot a trailing slash. i.e.: make KEYMAP_PATH=/foo/bar This patch makes usage of snprintf() and puts an explicit slash between KEYMAP_PATH and mapname. If you add a trailing slash, you'll have 2. But, this doesn't hurt someone, where a missing does. greetings, Daniel (Yes, the patch doesn't add any functionallity - just some beautifing.) ---------------------------------------------------------------------- >Comment By: Ilya Konstantinov (ikonst) Date: 2006-09-12 12:09 Message: Logged In: YES user_id=335423 Looks legit. Could someone merge this in? ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=381349&aid=1552007&group_id=24366 |