harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alexei Fedotov (JIRA)" <j...@apache.org>
Subject [jira] Updated: (HARMONY-5692) [drlvm][build] remove apr-util and apr-iconv from the list of dependencies
Date Mon, 14 Apr 2008 04:57:05 GMT

     [ https://issues.apache.org/jira/browse/HARMONY-5692?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Alexei Fedotov updated HARMONY-5692:
------------------------------------

    Attachment: logger.5.patch

Linux specific changes:
 * added std:: to interpreter ostringstream
 * removed <<dex and <<hex from interpreter logging (if needed, please create
operator<< for a specific type)
 * apr atomics implemented via mutex hashtable crashed: using port atomics now
 * vmcore.exp modified with new logger exports

A checklist for Alexey:
    - ability to enable/disable ANY category - rather than support only per-category control;
explained
    - ability to specify a list of categories (-Xtrace:init,jni,enum);
not done, multiple -Xtrace option should be used, requires ~7 lines including strtok to be
implemented, I just did not choose it for the first release
    - ability to suppress certain categories (e.g. trace all but GC and threading);
-Xnotrace, -Xnoverbose implemented and documented
    - would be nice to keep relations between logging levels (e.g. if tracing is enabled then
info is enabled as well);
not yet done, can be done with three lines of code (is_trace_enabled checks both ->trace
and ->info lists)
    - would be nice to support file per category - should not be too expensive.
not yet done, have no idea yet how to do this nicely
    - Please document changes in concepts, use cases, defaults - e.g. WARN now have no categories

WARN is not filtered but may print a category, all explanations are added to the resource
file
    - switching C++ logging to C is not always clearly reasonable and not quite consistent.
explained
    - port_nanotimer() is not suitable for timestamps, only for measuring time intervals.

replaced with clock()

At this moment I don't have an access to 64bit platforms.

> [drlvm][build] remove apr-util and apr-iconv from the list of dependencies 
> ---------------------------------------------------------------------------
>
>                 Key: HARMONY-5692
>                 URL: https://issues.apache.org/jira/browse/HARMONY-5692
>             Project: Harmony
>          Issue Type: Improvement
>          Components: DRLVM
>            Reporter: Alexei Fedotov
>         Attachments: logger.1.patch, logger.2.patch, logger.3.patch, logger.4.patch,
logger.5.patch, logger.patch, logger.test.patch
>
>
> Two optional apr modules (apr-util and apr-iconv) modules are not used by DRLVM. Removing
them speeds up a build process.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message