Oh yes, and thank you, I am happy we have so many people that take care about {OpenGrok
(and I hope we can somehow expedite the patches processing)

thnx
L

On 20.12.2012 16:44, Lubos Kosco wrote:

Deniz

to accept the patches under your name we would need an OCA # from you
you can get it by following this process:
http://hub.opensolaris.org/bin/view/Main/oca_faq
(it's just a plain email exchange, takes 2-3 days most, they will register you as contributor and assign a #)

the other way is that we will push it and just thank you in comments in case you give away the rights to your code (since those are small patches and not sure if they will be taken as they are - Knut, Trond ? )

and yes, I know it's PITA and I hope we will find a better way (Trond, how's the git history rewrite going on? ;) )

your choice
Lubos

On 20.12.2012 16:34, Deniz Türkoglu wrote:
OpenGrok prints the exception when it fails to find a file that
makes the results page quite 'dirty'. This patch checks for the
file's existence and skips it if it doesn't exist.

Signed-off-by: Deniz Turkoglu <[email protected]>
---
  src/org/opensolaris/opengrok/search/Results.java | 7 ++++---
  1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/org/opensolaris/opengrok/search/Results.java
b/src/org/opensolaris/opengrok/search/Results.java
index 7e0a0ae..8057bd3 100644
--- a/src/org/opensolaris/opengrok/search/Results.java
+++ b/src/org/opensolaris/opengrok/search/Results.java
@@ -207,9 +207,10 @@ public final class Results {
String htags = getTags(sh.sourceRoot, rpath, false);
out.write(sh.summerizer.getSummary(htags).toString());
                      } else {
-                        FileReader r = genre == Genre.PLAIN
-                                ? new FileReader(new
File(sh.sourceRoot, rpath))
-                                : null;
+                        File file = new File(sh.sourceRoot, rpath);
+                        FileReader r =
+                            (genre == Genre.PLAIN && file.exists()) ?
+                            new FileReader(file) : null;
sh.sourceContext.getContext(r, out, xrefPrefix,
                                  morePrefix, rpath, tags, true, null);
                      }
--
1.8.0.2
_______________________________________________
opengrok-dev mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/opengrok-dev


_______________________________________________
opengrok-dev mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/opengrok-dev

Reply via email to