Skip to content

Write Cache to file#44

Merged
ArquintL merged 7 commits into
viperproject:masterfrom
jogasser:master
Dec 7, 2021
Merged

Write Cache to file#44
ArquintL merged 7 commits into
viperproject:masterfrom
jogasser:master

Conversation

@jogasser

Copy link
Copy Markdown
Contributor
  • Added serialization & deserialization functionality to the ViperCache object
  • Added config option to specify a cache file

@ArquintL ArquintL self-requested a review December 6, 2021 08:44
Comment thread src/main/scala/viper/server/core/VerificationWorker.scala
Comment thread src/main/scala/viper/server/ViperConfig.scala Outdated
Comment thread src/main/scala/viper/server/core/ViperCache.scala Outdated
Comment thread src/main/scala/viper/server/core/ViperCache.scala
Comment thread src/main/scala/viper/server/core/ViperCache.scala Outdated
Comment thread src/main/scala/viper/server/core/ViperCache.scala
Comment thread src/main/scala/viper/server/core/ViperCache.scala Outdated
Comment thread src/main/scala/viper/server/frontends/http/ViperHttpServer.scala Outdated
Comment thread src/main/scala/viper/server/vsi/Cache.scala Outdated
Comment thread src/main/scala/viper/server/vsi/Cache.scala Outdated
@ArquintL

ArquintL commented Dec 6, 2021

Copy link
Copy Markdown
Member

Silicon PR #583 will fix the member lookup of errors in VerificationWorker but setting the scope field

Comment thread src/main/scala/viper/server/vsi/Cache.scala Outdated
Comment thread src/main/scala/viper/server/core/ViperCache.scala Outdated
Comment thread src/main/scala/viper/server/core/ViperCache.scala Outdated
Comment thread src/main/scala/viper/server/core/ViperCache.scala Outdated
Comment thread src/main/scala/viper/server/core/ViperCache.scala Outdated
Comment thread src/main/scala/viper/server/core/ViperCache.scala Outdated
Comment thread src/main/scala/viper/server/core/ViperCache.scala Outdated
jogasser and others added 3 commits December 7, 2021 10:14
@ArquintL

ArquintL commented Dec 7, 2021

Copy link
Copy Markdown
Member

The CI should be working again. Please pull from master

@ArquintL ArquintL left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good now. You can merge it

@jogasser

jogasser commented Dec 7, 2021

Copy link
Copy Markdown
Contributor Author

I don't have write access to merge this pull request

@ArquintL ArquintL merged commit 5907ce1 into viperproject:master Dec 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants