Re: Mistake assert tips in FST builder ?

2019-04-22 Thread Michael McCandless
Hello, Indeed, you cosmetic fix looks great -- I'll push that change. Thanks for noticing and raising! Mike McCandless http://blog.mikemccandless.com On Tue, Apr 16, 2019 at 12:04 AM zhenyuan wei wrote: > Hi, >With current newest version, 9.0.0-snapshot,In > Builder.UnCompileNode.addArc(

Re: Mistake assert tips in FST builder ?

2019-04-18 Thread Tomás Fernández Löbbe
The Lucene list is probably better for this question. I'd try java-u...@lucene.apache.org On Mon, Apr 15, 2019 at 9:04 PM zhenyuan wei wrote: > Hi, >With current newest version, 9.0.0-snapshot,In > Builder.UnCompileNode.addArc() function, > found this line: > > assert numArcs == 0 || label >

Mistake assert tips in FST builder ?

2019-04-15 Thread zhenyuan wei
Hi, With current newest version, 9.0.0-snapshot,In Builder.UnCompileNode.addArc() function, found this line: assert numArcs == 0 || label > arcs[numArcs-1].label: "arc[-1].label=" + arcs[numArcs-1].label + " new label=" + label + " numArcs=" + numArcs; Maybe assert tips is : assert numArcs ==