From: Marek Zawirski Date: Fri, 29 Aug 2008 12:18:47 GMT Subject: Re: [JGIT PATCH] Disambiguate "push not supported" from "repository not found" Message-ID: <48B7E927.2000205@gmail.com> In-Reply-To: <200808291120.44413.robin.rosenberg@dewire.com> Robin Rosenberg wrote: > fredagen den 29 augusti 2008 02.18.38 skrev Shawn O. Pearce: >> + if (avail.isEmpty()) >> + throw noRepository(); >> throw eof; >> } >> >> @@ -185,6 +182,10 @@ else if (this instanceof FetchConnection) >> available(avail); >> } >> >> + protected TransportException noRepository() { >> + return new NoRemoteRepositoryException(uri, "not found."); >> + } >> + > > Why an extra method for instantiating the exception? Isn't it overrode in subclass - BasePackPushConnection? -- Marek Zawirski [zawir] marek.zawirski@gmail.com