[Blotter-commits] r1619 - pkg/FinancialInstrument/R

[G See]

Never mind.  I didn't look close enough.  This is fine.

Thank you
Garrett

On Tue, Jul 8, 2014 at 12:31 PM, G See <gsee000 at gmail.com> wrote:
> You cannot do this due to CRAN policy.  The FinancialInstrument:::
> should not be necessary because this function belongs to the
> FinancialInstrument namespace.
>
> On Tue, Jul 8, 2014 at 8:50 AM,  <noreply at r-forge.r-project.org> wrote:
>> Author: signori
>> Date: 2014-07-08 15:50:56 +0200 (Tue, 08 Jul 2014)
>> New Revision: 1619
>>
>> Modified:
>>    pkg/FinancialInstrument/R/saveInstruments.R
>> Log:
>> In saveInstruments, 'pos' should 'FinancialInstrument:::.instrument' (reverts to r1475)
>>
>> Modified: pkg/FinancialInstrument/R/saveInstruments.R
>> ===================================================================
>> --- pkg/FinancialInstrument/R/saveInstruments.R 2014-07-03 18:19:59 UTC (rev 1618)
>> +++ pkg/FinancialInstrument/R/saveInstruments.R 2014-07-08 13:50:56 UTC (rev 1619)
>> @@ -75,7 +75,7 @@
>>              "require(FinancialInstrument)\n\n", file=file.name)
>>          for (s in ls_instruments()) {
>>              sink(file.name, append=TRUE)
>> -            cat('assign("', s, '", pos=.instrument, ',
>> +            cat('assign("', s, '", pos=FinancialInstrument:::.instrument, ',
>>                  'value=\n', sep="", append=TRUE)
>>              dput(getInstrument(s))
>>              cat(")\n\n", append=TRUE)
>>
>> _______________________________________________
>> Blotter-commits mailing list
>> Blotter-commits at lists.r-forge.r-project.org
>> https://lists.r-forge.r-project.org/cgi-bin/mailman/listinfo/blotter-commits