Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Describe design decisions and implementation reasons #2

Open
mnogu opened this issue Oct 28, 2012 · 0 comments
Open

Describe design decisions and implementation reasons #2

mnogu opened this issue Oct 28, 2012 · 0 comments
Milestone

Comments

@mnogu
Copy link
Member

mnogu commented Oct 28, 2012

Moved from https://bugs.freedesktop.org/show_bug.cgi?id=639

YamaKen 2004-05-18 01:15:22 UTC

Describe design decisions and implementation reasons is required for developers
to answer "Why behaves so?" and to prevent corruption of implicit specification
on code rewriting.

Comment 1 YamaKen 2004-05-19 05:43:45 UTC

I have described a new function to follow the subject. I will continue so for a
week as a test drive.

/** Update custom value from property message.

  • Update custom value from property message. All variable update is

  • validated by custom APIs rather than arbitrary sexp

  • evaluation. Custom symbol \a custom is quoted in sexp string to be

  • restricted to accept symbol literal only. This prevents arbitrary

  • sexp evaluation.
    */
    void
    uim_prop_update_custom(uim_context uc, const char *custom, const char *val)
    {
    if(!custom || !val)
    return;

    UIM_EVAL_FSTRING3(uc, "(prop-handler %d 'prop-update-custom '%s %s)",
    uc->id, custom, val);
    }

@NgoHuy NgoHuy mentioned this issue Jul 5, 2013
icy added a commit to icyfork/uim that referenced this issue Nov 26, 2014
Add default destination for libuim output instead of /plugins

Without DESTDIR , uim will use /plugins as output directory. 
Let set empty variable will make it use right output directory.
icy added a commit to icyfork/uim that referenced this issue Nov 26, 2014
This reverts commit 64480c4, reversing
changes made to 75c12fc.

Reason:

* Changes are only made for PKGBUILD (TheSLinux specific)
* There isn't a clear reason why we do need this change
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant