simplify mkSkolem naming system: don't use $$

Short summary: By default NODEID is appeneded, just continue doing what you
were, just don't add the _$$ at the end.

Long summary:

Before this commit there were four (yes!) ways to specify the names for new
skolems, which result in names as given below

1) mkSkolem("name", ..., SKOLEM_FLAG_DEFAULT)  -> "name_NODEID"
2) mkSkolem("name", ..., SKOLEM_EXACT_NAME)  -> "name"
3) mkSkolem("name_$$", ..., SKOLEM_FLAG_DEFAULT) -> "name_NODEID"
4) mkSkolem("na_$$_me", ..., SKOLEM_FLAG_DEFAULT) -> "na_NODEID_me"

After this commit, only 1) and 2) stay.

90% usage is of 1) or 3), which results in exact same behavior (and
looking at the source code it doesn't look like everyone realized that
the _$$ is just redundant).

Almost no one used 4), which is the only reason to even have $$. Post this
commit if you really want a number in the middle, manually construct the
name and use the SKOLEM_EXACT_NAME flag.

use internal skolem numbering

refactor .travis.yml

Merge pull request #24 from kbansal/sets-model

sets, misc

Minor change to better support parameterized partial/total kinds (for upcoming datatypes work).

Revert E-matching datatypes fix.

Handle fmf.card as input from user, add support in SMT2 parser, as requested by Martin Brain.  Fix two minor bugs : E-matching with datatypes, instantiation caching with incremental.

fix get-info error-behavior

add tests

fix

prep for fix

try foreach on CD datastructure

inputs to trigger bug

more

some debugging changes

Merge branch 'master' of https://github.com/CVC4/CVC4

Reduced example from pcc's bug report.

Merge pull request #21 from pcc/ite-fix

Fix for ite of >=64bit wide bitvectors with unconstrained condition.

fix for hiding prompt/header in shell, error-behavior options as in SMTLIB

For security, add --no-filesystem-access option, which disables SMT-LIB script access to filesystem.

Allow turning off the interactive prompt while in interactive mode.

Properly quote symbols in SMT-LIB printer.

Some incremental bugs for Boolean terms, fixed.  Thanks to Pantazis Deligiannis and Jeroen Ketema for discovering this issue.

Merge branch '1.3.x'

Fixing bug 552.  There was a bug when integers are made using a string with a lot of leading 0s on old versions of CLN.

Merge branch 'master' of github.com:tiliang/CVC4

windows build fix for UINT32_MAX

windows build fix for UINT32_MAX

Travis-CI test for new-theory script, also related bugfixes.

Merge branch 'master' of github.com:tiliang/CVC4

add str to u16/u32, and u16/u32 to str

minor typo/bug fix

add str to u16/u32, and u16/u32 to str

Merge pull request #23 from kbansal/sets-model

Sets model

rm old unused code

get-antlr error on missing config.guess

add construles, type_rules rm redundant, kinds cleanup

minor printer fix; intersection fix

Merge branch 'master' of github.com:tiliang/CVC4

Conflicts:
src/theory/strings/regexp_operation.cpp
src/theory/strings/regexp_operation.h

adds new feature: re.loop

adds intersection

deriv symbolic regexp

adds intersection

adds new feature: re.loop

Merge branch '1.3.x'

Conflicts:
src/theory/arith/normal_form.cpp

Win32 build script fixes (to allow portfolio builds).

Fix an off-the-end string pointer bug (showed up only in some Win32 builds).

Merge branch 'master' of github.com:tiliang/CVC4

deriv symbolic regexp

adds intersection

deriv symbolic regexp

Merging in a fix from 1.3.x.
Fixes an idempotency issue for non-linear multiplication with integer and real variables.

Conflicts:
src/theory/arith/normal_form.cpp

Fixes an idempotency issue for non-linear multiplication with integer and real variables.

adds intersection

Merge pull request #22 from kbansal/sets-model

Sets model

cleanup

fix for sets/mar2014/..317minimized..

Observed behavior:
  --check-model failed for set-term (union (z3f69 z3v151) (setenum z3v143))
with different set of elements in the model for representative and the node
itself.

Issue:
  The trouble with data structure being mainted to ensure that things
for which lemmas have been generated are not generated again. This
data structure (d_pendingEverInserted) needs to be user context
dependent. The bug was in the sequence of steps from requesting that
a lemma be generated to when it actually was. Sequence was:
addToPending (and also adds to pending ever inserted) ->
isComplete (might remove things from pending if requirment met in other ways) ->
getLemma (actually generated the lemma, if requirement not already met)

Resolution:
  adding terms to d_pendingEverInserted was moved from addToPending()
to getLemma().

Fix for registration issues of term appearing in a shared lemma

(brought to attention by lianah -- fix currently just adapted using
arrays -- this is to remind me to raise why do we even have this
isPreregistered bussiness)

rewriter fix, weaken an assertion

constant normal form and rewrite

fix a sharing issues with sets

push subtyping for sets to the element type

for eg, (Set Int) is subtype of (Set Real) if Int is subtype of Real

enable check-models for sets/ regressions

work on set model

testlemma regressions

Minor fix for CBQI, ignore inst constant nodes.

Fix documentation for Theory::preRegisterTerm().

Refactor the theory specific parts of definition expansion into the theory solvers.

In the process of doing this I may have fixed some bugs or some potential bugs so there
may be some user visible results of this refactoring.

Signed-off-by: Morgan Deters <mdeters@cs.nyu.edu>

Set dumping options from (set-option..) and API more directly.

Fix for bug 555; SMT-LIBv2 symbols now output with proper quoting.

Move the translator binary from src/main to examples, no longer built by default.

Appease compilers from latest XCode release (v5.1).

Minor usability fixes related to SMT-LIB compliance.

Fix proof signatures makefile

Minor documentation fixups.

Fix a memory leak in LFSC proof checker.  Largest QF_UF proof from Morgan (problematic-lfsc-pf) now successfully checks in 58 seconds using ~8 GB memory.  Add example test proof.

Merge branch 'master' of github.com:tiliang/CVC4

hot fix for pre-reg term caching in strings

hot fix for pre-reg term caching in strings

Fix for ite of >=64bit wide bitvectors with unconstrained condition.

SMT-LIB compliance: allow bin/hex set-info, e.g. (set-info :key #xffff).  Thanks to David Cok for the bug report.

dos2unix on the proof signatures, and fix the makefile.

Add ability to provide theory-specific proof rules to EqualityEngine, extends enumeration of MergeReasonType.  Add initial use in TheoryArrays.

Add working example of LFSC proof with quantifiers.  Update quantifiers signature to avoid dependent types in side condition.

Work on array pf signature, add working example.  Add quantifiers proof signature. Ignore terms not in current master EE for QCF.  Minor refactoring.  Make --rewrite-rules true by default.

Minor fixes post-merge of RR.

Fix LogicInfo unit test.

Some standardization of regression Makefiles that got out of sync.  Fixes cases of nonterminating rewrite-rules regressions.

Draft contrib/get-abc script for bitvectors libabc support.

Merge branch '1.3.x'

Fix for rewriterules build breakage.

Fix for random-seed option.

Fix for portfolio.

Minor cleanup.

* Reenable parts of bvsimple test
* Fix typo in #endif comment

Fix for (get-assignment), resolves bug 553.

Merge branch '1.3.x'

Fix some Win32 and SMT-LIB compliance bugs discovered by David Cok.

Initial refactor of rewrite rules, make theory_rewriterules empty theory.  Push kinds to quantifiers/kinds, rewrite rewrite rules in rewriter.

Fix rewrite rule attribute, refactor QCF, instantiation engine does not register rewrite rules, minor clean up.

QCF is now functional backend to rewrite rules. Support boolean variables in QCF.  Change check-model to ignore rewrite rules.  Incorporate some fixes from master.

Move rewrite of rewrite rules from rewriter to preprocessor, attribute becomes pattern.

Minor fixes to QCF/rewrite engine, removes RE on last call approach.  Add option for one inst per round in RE.

Merging rewrite rules into master, check-model current fails on 3 FMF regressions.

Fix check-model issue, a line of code was omitted during merge.

adds intro vars length cache

Merge branch 'master' of github.com:tiliang/CVC4