 |
 |
Daniel, please open a ticket, assign to one of sprints in S14 Jacek On 04/25/2014 12:46 PM, Wang, Daniel Liwei wrote: > Jacek, > It looks fishy and we'll fix it. There is live code in each. I had a > couple of ideas on how to fix it, but couldn't settle on one quickly and > so I decided to just leave it for now. > ParseException: decl for class ParseException > parseExceptions: decl for other exceptions arising from parsing. Right > now, they don't inherit from ParseException because ParseException is > conceptually defined on an ANTLR node (in theory, the exception catcher > could peek at the node and infer what went wrong), but the plumbing is > not there. > > If you have an idea for a cleaner solution, you are absolutely welcome > to go ahead and make it happen. > -Daniel > > On 04/25/2014 11:42 AM, Jacek Becla wrote: >> Daniel, >> >> Please look at QuerySession.cc: >> >> #include "parser/ParseException.h" >> #include "parser/parseExceptions.h" >> >> this looks fishy... >> >> Again, please don't touch the master until I am done. >> >> Jacek >> >> ######################################################################## >> Use REPLY-ALL to reply to list >> >> To unsubscribe from the QSERV-L list, click the following link: >> https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=QSERV-L&A=1 > ######################################################################## Use REPLY-ALL to reply to list To unsubscribe from the QSERV-L list, click the following link: https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=QSERV-L&A=1