Swordfish

Wow - I've just found one thing that really should go in the 'how to optimise your code' FAQ. I have a 'Program Globals.bas' in which I define all variables which need to be global rather than sub / function temporary. I've moved *all* variable definitions in to that - I had quite a few which I'd st...

Unfortunately neither of those work-around work in my scenario. I'm writing this const directly to EEPROM so using a LongInt would over-write data in another location. Using an integer value is also not ideal - for initial testing I'm using a whole number but I may need to tweak these defaults to us...

Thinking back, when I used to use the BKND compiler for the PIC12/16 series, that had the ability to specify which bank a particular variable was stored in. Does Swordfish offer a similar function? I guess that ideally the compiler would intelligently work out where to put variables to minimise bank...

The program I'm working on currently implements two serial buffers, one for receive and one for transmit. I was rather surprised when I increased the sizes from 16/64 bytes to 32/128 bytes that my code size also increased - by 133 words! After comparing the before and after .asm files it appears the...

I wonder if the issue is something as simple as the fact the compiler is assuming any integer const will be negative and sets the MSB regardless? The fault persists even with a simple Const as Integer = 3.0 declaration yet Const as Integer = -3.0 works as expected. I've PM'd David, hopefully it is s...

I don't see where is the incorrect behaviour. If you want that compiler generate integer arithmetic, you need to tag your expression with correct type (this is what you have done using AS WORD). Erm... So you think if I use Public Const ReadyThreshold1 As Integer = 3.0 * TemperatureMultiplier and t...

I think you'll find that using unsigned types vs signed types almost always produces smaller code if you can get away with it. How about in mixed expressions? Some values need to be signed - basically, I'm re-writing a PID routine that previously used floats. It seems that any const which would res...

To follow up on this, I've tried declaring this as an integer, a word and with no specific type. All three options end up with different compiled size with the 'no declaration' option being the biggest by 6 bytes. This const is only used in one place - it is written to EEPROM as a 'default' configur...

ReadyThreshold1 is actually -32576... That's "192 with the MSB set", turning it into a negative number Drop the 'as integer' from the declaration and it works. It'll convert it to an int when it uses it. I'm not so confident to do that - from an earlier post you'll see that in a similar situation t...

I've found an oddity with consts in the compiler. Not sure if this is a bug or something that is counter-intuitive... If you enter the following code: Public Const TemperatureMultiplier = 64 // All temperatures are multiplied by 64 to allow the use of a signed integer rather than floats Public Const...

Maybe worth going to fixed point, I'll investigate. I actually only need 2dp, I'd gone to three to try and understand what was happening. I've found a sort of fix. If I add 0.006 that seems to give the expected results - after all, 123.125 should really output 123.13 if you are rounding to the neare...

I'm hitting an issue when outputting a float that the result appears to be suffering from rounding errors. A value of 123.125 will output correctly, however, 123.12 will output as 123.119. I'm guessing this will be due to the way numbers are represented, but I'm wondering is there any way to avoid t...

Thanks for trying, Jerry. All assumptions seem valid and your code compiles for me as well. It must be just one of those odd things - I couldn't find a way to replicate it without posting my whole code so I've done a work-around and moved the routine to copy the suffix in to a sub and that works fin...

I've got two functions which do the same thing but on different data types: Sub SendData(Suffix As String, Data As Word) FTextAddress = @TStr // Initialise pointer to start of the string FTextIn = Suffix(0) // Put the suffix as the first characters FTextIn = Suffix(1) FTextIn = GH_ETB // Send end of...

Thanks, Jerry, that is all I could find as well. I can't easily post the full code and can't seem to replicate it by cutting it down. I think it must just be one of those cases where the compiler does something odd which isn't repeatable. This was a sort of 'compound' constant. I had one constant I'...