* [EGIT PATCH] Showing abbreviated commit hash of the versions in Compare editor.
@ 2008-01-08 1:15 Roger C. Soares
2008-01-08 22:12 ` Robin Rosenberg
0 siblings, 1 reply; 9+ messages in thread
From: Roger C. Soares @ 2008-01-08 1:15 UTC (permalink / raw
To: git
Signed-off-by: Roger C. Soares <rogersoares@intelinet.com.br>
---
Some more patches, please evaluate.
Don't know what others are working on,
but right now I'm missing a search functionality so
this will probably be my next egit little feature.
cheers
.../GitCompareFileRevisionEditorInput.java | 12 ++++++++++++
.../spearce/egit/ui/internal/GitResourceNode.java | 5 +++++
2 files changed, 17 insertions(+), 0 deletions(-)
diff --git a/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/GitCompareFileRevisionEditorInput.java b/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/GitCompareFileRevisionEditorInput.java
index eaba1fa..403fdc1 100644
--- a/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/GitCompareFileRevisionEditorInput.java
+++ b/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/GitCompareFileRevisionEditorInput.java
@@ -152,6 +152,18 @@ public class GitCompareFileRevisionEditorInput extends CompareEditorInput {
private void initLabels(ICompareInput input) {
CompareConfiguration cc = getCompareConfiguration();
+ if(left != null && left instanceof GitResourceNode) {
+ String ci = ((GitResourceNode)left).getContentIdentifier();
+ if(ci != null) {
+ cc.setLeftLabel(ci.substring(0, 7) + "..");
+ }
+ }
+ if(right != null && right instanceof GitResourceNode) {
+ String ci = ((GitResourceNode)right).getContentIdentifier();
+ if(ci != null) {
+ cc.setRightLabel(ci.substring(0, 7) + "..");
+ }
+ }
if (getLeftRevision() != null) {
String leftLabel = getFileRevisionLabel(getLeftRevision());
cc.setLeftLabel(leftLabel);
diff --git a/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/GitResourceNode.java b/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/GitResourceNode.java
index e19cef5..f25855a 100644
--- a/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/GitResourceNode.java
+++ b/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/GitResourceNode.java
@@ -21,6 +21,7 @@ import org.spearce.jgit.lib.TreeEntry;
public class GitResourceNode extends BufferedContent implements IStructureComparator, ITypedElement {
TreeEntry entry;
GitResourceNode[] children;
+ String contentIdentifier;
public GitResourceNode(TreeEntry e) {
entry = e;
@@ -28,6 +29,7 @@ public class GitResourceNode extends BufferedContent implements IStructureCompar
public GitResourceNode(IFileRevision file) {
this(file instanceof GitCommitFileRevision ? ((GitCommitFileRevision)file).getTreeEntry() : null);
+ contentIdentifier = ((GitCommitFileRevision)file).getContentIdentifier();
}
public Object[] getChildren() {
@@ -100,5 +102,8 @@ public class GitResourceNode extends BufferedContent implements IStructureCompar
}
}
+ public String getContentIdentifier() {
+ return contentIdentifier;
+ }
}
--
1.5.3.7
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [EGIT PATCH] Showing abbreviated commit hash of the versions in Compare editor.
2008-01-08 1:15 [EGIT PATCH] Showing abbreviated commit hash of the versions in Compare editor Roger C. Soares
@ 2008-01-08 22:12 ` Robin Rosenberg
2008-01-09 1:33 ` Roger C. Soares
0 siblings, 1 reply; 9+ messages in thread
From: Robin Rosenberg @ 2008-01-08 22:12 UTC (permalink / raw
To: Roger C. Soares; +Cc: git, Shawn O. Pearce
Thanks.
tisdagen den 8 januari 2008 skrev Roger C. Soares:
>
> Signed-off-by: Roger C. Soares <rogersoares@intelinet.com.br>
> ---
> Some more patches, please evaluate.
> Don't know what others are working on,
I'm implementing fetch (with some help from Shawn). Progress in
place. I still need to update refs and get in consistent with native
git behaviour.
I'm also updating the javadocs.
> but right now I'm missing a search functionality so
> this will probably be my next egit little feature.
Sounds good.
-- robin
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [EGIT PATCH] Showing abbreviated commit hash of the versions in Compare editor.
2008-01-08 22:12 ` Robin Rosenberg
@ 2008-01-09 1:33 ` Roger C. Soares
2008-01-10 23:46 ` Robin Rosenberg
0 siblings, 1 reply; 9+ messages in thread
From: Roger C. Soares @ 2008-01-09 1:33 UTC (permalink / raw
To: Robin Rosenberg; +Cc: git, Shawn O. Pearce
Robin Rosenberg escreveu:
> I'm implementing fetch (with some help from Shawn). Progress in
> place. I still need to update refs and get in consistent with native
> git behaviour.
>
> I'm also updating the javadocs.
Cool.
I use a lot the eclipse feature that highlights all occurences of a
variable or method but with all the warnings in the project it is
difficult to visually find these occurences. So I'm anxious to remove a
lot of warnings. Public methods without documentation are one of them.
Is it ok if I send you a patch removing all the warnings I know how but
the ones about public methods needing documentation as you're already
documenting them? I also want to remove the auto generated TODOs, as
they don't actually say what to do, like this one:
// TODO Auto-generated catch block
[]s,
Roger.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [EGIT PATCH] Showing abbreviated commit hash of the versions in Compare editor.
2008-01-09 1:33 ` Roger C. Soares
@ 2008-01-10 23:46 ` Robin Rosenberg
2008-01-12 3:02 ` Roger C. Soares
2008-01-12 3:37 ` Roger C. Soares
0 siblings, 2 replies; 9+ messages in thread
From: Robin Rosenberg @ 2008-01-10 23:46 UTC (permalink / raw
To: Roger C. Soares; +Cc: git, Shawn O. Pearce
onsdagen den 9 januari 2008 skrev Roger C. Soares:
> Robin Rosenberg escreveu:
> > I'm implementing fetch (with some help from Shawn). Progress in
> > place. I still need to update refs and get in consistent with native
> > git behaviour.
> >
> > I'm also updating the javadocs.
> Cool.
>
> I use a lot the eclipse feature that highlights all occurences of a
> variable or method but with all the warnings in the project it is
> difficult to visually find these occurences. So I'm anxious to remove a
> lot of warnings. Public methods without documentation are one of them.
> Is it ok if I send you a patch removing all the warnings I know how but
> the ones about public methods needing documentation as you're already
> documenting them? I also want to remove the auto generated TODOs, as
> they don't actually say what to do, like this one:
> // TODO Auto-generated catch block
I pushed my doc-branch as pu, you can look at it. My intention is take a look again
and then push it as master if there are no objections. Is it ok and good to flags
public stuff without javadoc as an error (tip pu commit)?
The abbr commit patch is already pushed to master.
-- robin
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [EGIT PATCH] Showing abbreviated commit hash of the versions in Compare editor.
2008-01-10 23:46 ` Robin Rosenberg
@ 2008-01-12 3:02 ` Roger C. Soares
2008-01-12 3:37 ` Roger C. Soares
1 sibling, 0 replies; 9+ messages in thread
From: Roger C. Soares @ 2008-01-12 3:02 UTC (permalink / raw
To: Robin Rosenberg; +Cc: git, Shawn O. Pearce
Robin Rosenberg escreveu:
> I pushed my doc-branch as pu, you can look at it. My intention is take a look again
> and then push it as master if there are no objections. Is it ok and good to flags
> public stuff without javadoc as an error (tip pu commit)?
>
> The abbr commit patch is already pushed to master.
>
Looks good to me. You actually took some time to explain things, it will
be usefull, thanks!
If we really want to document all the public methods then I think it's
nice to flag it's absence as an error. It would have made me document
GitResourceNode.getContentIdentifier(), it's now one more comment for
you to add before pushing to master ;)
[]s,
Roger.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [EGIT PATCH] Showing abbreviated commit hash of the versions in Compare editor.
2008-01-10 23:46 ` Robin Rosenberg
2008-01-12 3:02 ` Roger C. Soares
@ 2008-01-12 3:37 ` Roger C. Soares
2008-01-12 15:44 ` Robin Rosenberg
1 sibling, 1 reply; 9+ messages in thread
From: Roger C. Soares @ 2008-01-12 3:37 UTC (permalink / raw
To: Robin Rosenberg; +Cc: git, Shawn O. Pearce
Robin Rosenberg escreveu:
> I pushed my doc-branch as pu, you can look at it. My intention is take a look again
> and then push it as master if there are no objections. Is it ok and good to flags
> public stuff without javadoc as an error (tip pu commit)?
>
Got some errors, when trying to export to a deployable plugin:
"/home/roger/eclipse/workspace/egit/org.spearce.jgit/tst" does not exist!
And on "team -> share project... -> Git", it goes to the CVS wizard.
[]s,
Roger.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [EGIT PATCH] Showing abbreviated commit hash of the versions in Compare editor.
2008-01-12 3:37 ` Roger C. Soares
@ 2008-01-12 15:44 ` Robin Rosenberg
2008-01-13 21:02 ` Roger C. Soares
0 siblings, 1 reply; 9+ messages in thread
From: Robin Rosenberg @ 2008-01-12 15:44 UTC (permalink / raw
To: Roger C. Soares; +Cc: git, Shawn O. Pearce
lördagen den 12 januari 2008 skrev Roger C. Soares:
>
> Robin Rosenberg escreveu:
> > I pushed my doc-branch as pu, you can look at it. My intention is take a look again
> > and then push it as master if there are no objections. Is it ok and good to flags
> > public stuff without javadoc as an error (tip pu commit)?
> >
> Got some errors, when trying to export to a deployable plugin:
> "/home/roger/eclipse/workspace/egit/org.spearce.jgit/tst" does not exist!
>
> And on "team -> share project... -> Git", it goes to the CVS wizard.
Due to my lack of testing @_@. I pushed a new pu branch with these two errors fixed.
Thanks
-- robin
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [EGIT PATCH] Showing abbreviated commit hash of the versions in Compare editor.
2008-01-12 15:44 ` Robin Rosenberg
@ 2008-01-13 21:02 ` Roger C. Soares
2008-01-13 23:54 ` javadoc (was [EGIT PATCH] Showing abbreviated commit hash of the versions in Compare editor.) Robin Rosenberg
0 siblings, 1 reply; 9+ messages in thread
From: Roger C. Soares @ 2008-01-13 21:02 UTC (permalink / raw
To: Robin Rosenberg; +Cc: git, Shawn O. Pearce
Hi Robin,
For what I've been using it's all working now.
I saw some typos if you want to update them while you're at it:
Activator.java
* Instatiate an error exception.
* Log a dbeug message
GitHistoryPage.java
* @return reformated comment
* This class extendes the build in history page with Git features, such
[]s,
Roger.
--
Robin Rosenberg escreveu:
> lördagen den 12 januari 2008 skrev Roger C. Soares:
>
>> Robin Rosenberg escreveu:
>>
>>> I pushed my doc-branch as pu, you can look at it. My intention is take a look again
>>> and then push it as master if there are no objections. Is it ok and good to flags
>>> public stuff without javadoc as an error (tip pu commit)?
>>>
>>>
>> Got some errors, when trying to export to a deployable plugin:
>> "/home/roger/eclipse/workspace/egit/org.spearce.jgit/tst" does not exist!
>>
>> And on "team -> share project... -> Git", it goes to the CVS wizard.
>>
>
> Due to my lack of testing @_@. I pushed a new pu branch with these two errors fixed.
>
> Thanks
> -- robin
>
>
>
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: javadoc (was [EGIT PATCH] Showing abbreviated commit hash of the versions in Compare editor.)
2008-01-13 21:02 ` Roger C. Soares
@ 2008-01-13 23:54 ` Robin Rosenberg
0 siblings, 0 replies; 9+ messages in thread
From: Robin Rosenberg @ 2008-01-13 23:54 UTC (permalink / raw
To: Roger C. Soares; +Cc: git, Shawn O. Pearce
söndagen den 13 januari 2008 skrev Roger C. Soares:
> Hi Robin,
>
> For what I've been using it's all working now.
> I saw some typos if you want to update them while you're at it:
Fixed, on pu now. Will push to master tomorrow.
-- robiin
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2008-01-13 23:55 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-01-08 1:15 [EGIT PATCH] Showing abbreviated commit hash of the versions in Compare editor Roger C. Soares
2008-01-08 22:12 ` Robin Rosenberg
2008-01-09 1:33 ` Roger C. Soares
2008-01-10 23:46 ` Robin Rosenberg
2008-01-12 3:02 ` Roger C. Soares
2008-01-12 3:37 ` Roger C. Soares
2008-01-12 15:44 ` Robin Rosenberg
2008-01-13 21:02 ` Roger C. Soares
2008-01-13 23:54 ` javadoc (was [EGIT PATCH] Showing abbreviated commit hash of the versions in Compare editor.) Robin Rosenberg
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
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).