Adding new cameras, applying patches into trunk (with source code prepared)

  • 1581 Replies
  • 228734 Views
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #1570 on: 02 / September / 2017, 16:59:45 »
Advertisements
Another zoom related patch for the ixus300 

(ref https://chdk.setepontos.com/index.php?topic=13160.msg134493#msg134493)
Ported : A1200  SD940  G10  Powershot N  G16*

*

Offline reyalp

  • ******
  • 10835
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #1571 on: 02 / September / 2017, 17:36:27 »
Another zoom related patch for the ixus300 

(ref https://chdk.setepontos.com/index.php?topic=13160.msg134493#msg134493)
Added in trunk 4906, release 4907
Don't forget what the H stands for.

Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #1572 on: 04 / September / 2017, 06:35:07 »
New Italian language update.

https://chdk.setepontos.com/index.php?topic=13240.new#new

Please include it in next release.

Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #1573 on: 04 / September / 2017, 11:33:43 »
Fix for two grid issues reported here :  Re: Grids : issues and challenges

  • Grid title from @title directive in .grd file now appears under Current Grid in the Grid Lines menu.  If no @name directive exists, the file name is used.
  • Fixed crashing due to the grid module unloading when the user selects a grid files while Show Grid Lines [ ] is disabled.
Ported : A1200  SD940  G10  Powershot N  G16*


*

Offline reyalp

  • ******
  • 10835
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #1574 on: 04 / September / 2017, 18:20:58 »
Fix for two grid issues reported here :  Re: Grids : issues and challenges
Thanks. Added in trunk 4911-4912, release 4913.

I changed the fix for the crash make it self contained in the module.
Don't forget what the H stands for.

Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #1575 on: 04 / September / 2017, 18:34:25 »
I changed the fix for the crash make it self contained in the module.
That was my original fix too. But then I wondered about a race condition where the module gets unloaded before grid_loading = 1 is executed?

Ported : A1200  SD940  G10  Powershot N  G16*

*

Offline reyalp

  • ******
  • 10835
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #1576 on: 04 / September / 2017, 18:51:05 »
That was my original fix too. But then I wondered about a race condition where the module gets unloaded before grid_loading = 1 is executed?
In theory maybe, but that's how most of the other modules with a "running" state work. If we need to resolve that, we should add some kind of delay or to the module_tick_unloader process.

I think in practice the window for problems there is nanoseconds, where the original grid loading could take tens of milliseconds and involved blocking IO calls that would yield to other tasks.
Don't forget what the H stands for.

Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #1577 on: 04 / September / 2017, 18:53:06 »
I think in practice the window for problems there is nanoseconds, where the original grid loading could take tens of milliseconds and involved blocking IO calls that would yield to other tasks.
Good enough.  I noticed that the bug occurred much more frequently on my older cameras and also with larger files.  All leading me to the race condition conclusion.
Ported : A1200  SD940  G10  Powershot N  G16*


Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #1578 on: 05 / September / 2017, 17:22:24 »
Hello!
Attached is an updated lang file for Polish language.
Thanks!
if (2*b || !2*b) {
    cout<<question
}

Compile error: poor Yorick

*

Offline reyalp

  • ******
  • 10835
Re: Adding new cameras, applying patches into trunk (with source code prepared)
« Reply #1579 on: 05 / September / 2017, 23:04:12 »
Hello!
Attached is an updated lang file for Polish language.
Thanks!
Thanks. Is the ?? intended in #258?
Code: [Select]
258 "W??czenie Lua Native Calls pozwala\numozliwia uszkodzenie aparatu.\nJesli nie jestes pewien\nNIE WLACZAJ TEJ OPCJI.\nJestes pewien wlaczyc Native Calls?"
I thought it might be an encoding issue, but it looks like it's really ASCII '?' in the file. (The file should be in windows 1250 encoding)
Don't forget what the H stands for.

 

Related Topics