Am 04.03.2016 um 14:40 schrieb tony mancill: [...] >> Hi tony, >> >> I can upload a new revision of bsh with this change later. I'm just wondering >> why we need to use java.io.ObjectStreamException and >> java.io.NotSerializableException explicitly because these classes are already >> imported in bsh's XThis.java. >> >> import java.io.*; >> >> Anyway it doesn't change the intention of the patch and should be safe. > > Hi Markus, > > I ask myself that same question, and intend to look it up. Perhaps > because it's a nested inner class of XThis? > > But one of the clues was that the original code specified the full class > name for java.io.Serializable and not just Serializable. > > I agree that it shouldn't affect the intention of patch and should be > safe, but it is a bit puzzling.
I have just uploaded bsh with this change. Cheers, Markus
signature.asc
Description: OpenPGP digital signature