fixed use of constructor