Bug Report #3032

Session's constructor shouldn't be protected

Added by Matt Button over 4 years ago. Updated over 4 years ago.

Status:ClosedStart date:06/29/2010
Priority:NormalDue date:
Assignee:Woody Gilk% Done:

0%

Category:Core
Target version:v3.0.7
Resolution:fixed Points:

Description

The class is already abstract, having a protected constructor just makes it harder to test.
I've implemented this in my fork and sent a pull request

Associated revisions

Revision 6bd1b3ca
Added by Matt Button over 4 years ago

Modifying Session to use public constructor. Class is already abstract,
this makes it easier to test.
Fixes #3032

History

#2 Updated by Woody Gilk over 4 years ago

  • Status changed from New to Closed
  • Target version changed from v3.0.6 to v3.0.7
  • Resolution set to fixed

Also available in: Atom PDF