public inbox for frysk-cvs@sourceware.org help / color / mirror / Atom feed
From: pmachata@sourceware.org To: frysk-cvs@sourceware.org Subject: [SCM] master: Coding style fix Date: Tue, 01 Apr 2008 15:54:00 -0000 [thread overview] Message-ID: <20080401155433.15493.qmail@sourceware.org> (raw) The branch, master has been updated via ff5c661252807c07e2bd8a43b0bb10a063aa6604 (commit) via 819139c2799329e30ef27c11e01f331c1dd213ec (commit) via c6a2ca1a192555d879a65ae6163ec6e851122447 (commit) via 6ecd11f916303cbae9ee327f233940de76d6bf6c (commit) from 164a248e5e4f8d6cf95d0ffcb2e58ea2b96bdf5a (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email. - Log ----------------------------------------------------------------- commit ff5c661252807c07e2bd8a43b0bb10a063aa6604 Author: Petr Machata <pmachata@redhat.com> Date: Tue Apr 1 16:34:23 2008 +0200 Coding style fix commit 819139c2799329e30ef27c11e01f331c1dd213ec Author: Petr Machata <pmachata@redhat.com> Date: Mon Mar 31 18:12:50 2008 +0200 Coding style fix commit c6a2ca1a192555d879a65ae6163ec6e851122447 Author: Petr Machata <pmachata@redhat.com> Date: Fri Mar 28 16:18:20 2008 +0100 Coding style fix commit 6ecd11f916303cbae9ee327f233940de76d6bf6c Author: Petr Machata <pmachata@redhat.com> Date: Fri Mar 21 12:28:55 2008 +0100 Reindent Symbol.java ----------------------------------------------------------------------- Summary of changes: frysk-core/frysk/dwfl/DwflCache.java | 30 +++--- frysk-core/frysk/ftrace/Rule.java | 2 +- frysk-core/frysk/ftrace/Symbol.java | 149 ++++++++++++------------- frysk-core/frysk/stepping/SteppingEngine.java | 6 +- 4 files changed, 90 insertions(+), 97 deletions(-) First 500 lines of diff: diff --git a/frysk-core/frysk/dwfl/DwflCache.java b/frysk-core/frysk/dwfl/DwflCache.java index 6f70aa0..15ffde3 100644 --- a/frysk-core/frysk/dwfl/DwflCache.java +++ b/frysk-core/frysk/dwfl/DwflCache.java @@ -77,21 +77,21 @@ public class DwflCache { */ private static WeakHashMap allDwfls = new WeakHashMap(); -/** - * Given: a mock generated /sys/root/dir/usr/bin/program - * /sys/root/dir/usr/lib/debug/usr/bin/program.debug - * Elfutils can be given an absolute or relative path to look for program.debug. - * It will look in: - * 1. /sys/root/dir/usr/bin/program.debug - * 2. /sys/root/dir/usr/bin/a/relative/path/program.debug - * 3. /an/absolute/path/sys/root/dir/usr/bin/program.debug - * It would be helpful if it also looked in /an/absolute/path/program.debug - * so it could be given /sys/root/dir/usr/lib/debug/usr/bin. Lacking that we - * need to generate a relative path that has the same effect. - * - * @param pathname of executable - * @return a path where elfutils can find program.debug for separate debuginfo. - */ + /** + * Given: a mock generated /sys/root/dir/usr/bin/program + * /sys/root/dir/usr/lib/debug/usr/bin/program.debug + * Elfutils can be given an absolute or relative path to look for program.debug. + * It will look in: + * 1. /sys/root/dir/usr/bin/program.debug + * 2. /sys/root/dir/usr/bin/a/relative/path/program.debug + * 3. /an/absolute/path/sys/root/dir/usr/bin/program.debug + * It would be helpful if it also looked in /an/absolute/path/program.debug + * so it could be given /sys/root/dir/usr/lib/debug/usr/bin. Lacking that we + * need to generate a relative path that has the same effect. + * + * @param pathname of executable + * @return a path where elfutils can find program.debug for separate debuginfo. + */ private static File getRelativeSysRoot(String execPathParm, File sysroot) { if (sysroot.getPath().equals("/")) diff --git a/frysk-core/frysk/ftrace/Rule.java b/frysk-core/frysk/ftrace/Rule.java index 8d6a60c..4745c6c 100644 --- a/frysk-core/frysk/ftrace/Rule.java +++ b/frysk-core/frysk/ftrace/Rule.java @@ -62,7 +62,7 @@ public abstract class Rule { } public boolean apply(Collection candidates, - Set workingSet, Set stackTraceSet) + Set workingSet, Set stackTraceSet) { boolean matched = false; diff --git a/frysk-core/frysk/ftrace/Symbol.java b/frysk-core/frysk/ftrace/Symbol.java index 050f95e..febdd3f 100644 --- a/frysk-core/frysk/ftrace/Symbol.java +++ b/frysk-core/frysk/ftrace/Symbol.java @@ -59,74 +59,69 @@ public class Symbol * alternative names. */ public ArrayList aliases = null; - public final long value; - public final long size; + public final long value; + public final long size; public final long offset; // Relative to ELF file start. - public final ElfSymbolType type; - public final long shndx; - - public final ElfSymbolVersion.Def[] verdefs; - public final ElfSymbolVersion.Need[] verneeds; - - protected ObjectFile parent = null; - - /** - * Build ltrace symbol. - * - * @param name Name of the symbol. - * @param type Type of the symbol, as in ElfSymbol.ELF_STT_* fields. - * @param value Value of the symbol. - * @param size Size of the symbol. - * @param shndx Associated section index, or one of the special - * values in ElfSectionHeader.ELF_SHN_*. - * @param versions Version requirements and/or definitions of - * symbol. If there are none, null is passed instead of - * zero-length array. - */ - public Symbol(final String name, ElfSymbolType type, long value, - long offset, long size, long shndx, List versions) - { - this.name = name; - this.aliases = null; - this.type = type; - this.value = value; - this.offset = offset; - this.size = size; - this.shndx = shndx; - - final ArrayList foundDefs = new ArrayList(); - final ArrayList foundNeeds = new ArrayList(); - - if (versions != null) - for (Iterator it = versions.iterator(); it.hasNext(); ) - ((ElfSymbolVersion)it.next()).visit(new ElfSymbolVersion.Visitor() { - public Object def(ElfSymbolVersion.Def verdef) { - foundDefs.add(verdef); - return null; - } - public Object need(ElfSymbolVersion.Need verneed) { - foundNeeds.add(verneed); - return null; - } - }); - + public final ElfSymbolType type; + public final long shndx; + + public final ElfSymbolVersion.Def[] verdefs; + public final ElfSymbolVersion.Need[] verneeds; + + protected ObjectFile parent = null; + + /** + * Build ltrace symbol. + * + * @param name Name of the symbol. + * @param type Type of the symbol, as in ElfSymbol.ELF_STT_* fields. + * @param value Value of the symbol. + * @param size Size of the symbol. + * @param shndx Associated section index, or one of the special + * values in ElfSectionHeader.ELF_SHN_*. + * @param versions Version requirements and/or definitions of + * symbol. If there are none, null is passed instead of + * zero-length array. + */ + public Symbol(final String name, ElfSymbolType type, long value, + long offset, long size, long shndx, List versions) { - this.verdefs = new ElfSymbolVersion.Def[foundDefs.size()]; - int i = 0; - for (Iterator it = foundDefs.iterator(); it.hasNext(); ) - this.verdefs[i++] = (ElfSymbolVersion.Def)it.next(); + this.name = name; + this.aliases = null; + this.type = type; + this.value = value; + this.offset = offset; + this.size = size; + this.shndx = shndx; + + final ArrayList foundDefs = new ArrayList(); + final ArrayList foundNeeds = new ArrayList(); + + if (versions != null) + for (Iterator it = versions.iterator(); it.hasNext(); ) + ((ElfSymbolVersion)it.next()).visit(new ElfSymbolVersion.Visitor() { + public Object def(ElfSymbolVersion.Def verdef) { + foundDefs.add(verdef); + return null; + } + public Object need(ElfSymbolVersion.Need verneed) { + foundNeeds.add(verneed); + return null; + } + }); + + int i = 0; + this.verdefs = new ElfSymbolVersion.Def[foundDefs.size()]; + for (Iterator it = foundDefs.iterator(); it.hasNext(); ) + this.verdefs[i++] = (ElfSymbolVersion.Def)it.next(); + + i = 0; + this.verneeds = new ElfSymbolVersion.Need[foundNeeds.size()]; + for (Iterator it = foundNeeds.iterator(); it.hasNext(); ) + this.verneeds[i++] = (ElfSymbolVersion.Need)it.next(); } - { - this.verneeds = new ElfSymbolVersion.Need[foundNeeds.size()]; - int i = 0; - for (Iterator it = foundNeeds.iterator(); it.hasNext(); ) - this.verneeds[i++] = (ElfSymbolVersion.Need)it.next(); - } - } - - public void addAlias(String alias) - { + public void addAlias(String alias) { if (this.aliases == null) this.aliases = new ArrayList(); @@ -140,17 +135,15 @@ public class Symbol this.aliases.add(alias); } - public String toString() - { - StringBuffer buf = new StringBuffer(); - buf.append(this.name); - return buf.toString(); - } + public String toString() { + StringBuffer buf = new StringBuffer(); + buf.append(this.name); + return buf.toString(); + } /** Answer true, if name of the symbol, or one of the aliases, * match given NAME. */ - public boolean hasName(String name) - { + public boolean hasName(String name) { if (this.name.equals(name)) return true; @@ -162,11 +155,11 @@ public class Symbol return false; } - public void addedTo(ObjectFile of) { - this.parent = of; - } + public void addedTo(ObjectFile of) { + this.parent = of; + } - public ObjectFile getParent() { - return this.parent; - } + public ObjectFile getParent() { + return this.parent; + } } diff --git a/frysk-core/frysk/stepping/SteppingEngine.java b/frysk-core/frysk/stepping/SteppingEngine.java index 6fa9b3e..7bb4ba4 100644 --- a/frysk-core/frysk/stepping/SteppingEngine.java +++ b/frysk-core/frysk/stepping/SteppingEngine.java @@ -1119,10 +1119,10 @@ public class SteppingEngine { list.add(t); Proc proc = t.getProc(); - if (!(proc.getUID() == Manager.host.getSelf().getUID() || proc - .getGID() == Manager.host.getSelf().getGID())) { + if (!(proc.getUID() == Manager.host.getSelf().getUID() + || proc.getGID() == Manager.host.getSelf().getGID())) { System.err.println("Process " + proc - + " is not owned by user/group."); + + " is not owned by user/group."); continue; } } hooks/post-receive -- frysk system monitor/debugger
next reply other threads:[~2008-04-01 15:54 UTC|newest] Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top 2008-04-01 15:54 pmachata [this message] 2008-04-02 16:53 pmachata
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20080401155433.15493.qmail@sourceware.org \ --to=pmachata@sourceware.org \ --cc=frysk-cvs@sourceware.org \ --cc=frysk@sourceware.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).