Oh, though it may depend on your definition of "register a device". For clarity: providing a new android_id and logging in will register a Google Accounts device (eg https://github.com/simon-weber/gpsoauth/issues/2). If you never stream music with that device (ie, call get_stream_url), it will _not_ be registered as a Google Music device, and should not count against your device limit.

As far as I know, there's no limit to Google Accounts devices, but I haven't really looked for one.


I don't understand what is changed: with release 5.0.0 I can login and query the registered devices without inserting any android_id...


closed #61, closes #24


Трэба зрабіць лагарыфмічны маштаб - інакш нічога не зразумела..


Fixed in https://github.com/Homebrew/homebrew/commit/d4dbd62fbf411d901f9933cb5e18402dde287eb1.


Correct; this change is pending a future release. Not providing an android_id makes Google register a new Accounts device every time, which doesn't seem good; check out the linked gpsoauth issue for an example.


Атрымліваецца, што вектар лепей за ўсіх?


This stops functionality to use expressions in addCondition ???
It should be continued as while it can and was working good.

Now it is forced to use $model->_dsql()->having() to use for expression ... All Codes Broken !!!


:innocent:


I'm sorry, but why on earth would you backslash every slash in strings? This makes no sense, it's not like a regex literal there.


fix #366


What does this mean? What should be fixed here?


Instead of hosting my CV on my site, I keep the .pages and .pdf in my Dropbox public folder and link to the .pdf on my website. This means whenever I revise the CV, it automatically syncs with Dropbox, but the URL remains the same. So my CV is always up to date without the additional step of uploading the revised version to Github pages.

You can achieve a similar but slightly less automated workflow by keeping the .pages and .pdf in your repo so the latest version is always pushed when you push rest of changes.


Хацелася такі  пабачыць больше за адзін радок...


Then you can put his name, or email too, that works sometimes
Its ok now :P


Bah on est noté sur la propreté du code... moins que sur ces fonctionnalités je crois


@zentuit why did you add this `postKeyValueStorageChange` ?
It's here by mistake right?


commit message should be "added range checking"


FWIW I made a tweak that allows the command to work on OSX (BSD sed) and non-OSX (assumes GNU sed): https://github.com/justinmk/config/commit/919eec6700903365525c04f30b61ec92bc36c955#diff-85cf9b51417c7cfb8766aa6b56f7edb9R275


І што там атрымалась у гэтым list.csv?


Hi @jypma. Sry it's  long ago. I can only guess.
Maybe it was becaus of implementing a non-Ascii based protocol ... but ... no: I really can't remember.


@zentuit does this also solves the recurring login window at the beginning?

I don't understand why this doesn't work automatically... we add the observer early in the initialization process.


Creative Commons Attribution-NonCommercial 4.0 International Public License

By exercising the Licensed Rights (defined below), You accept and agree to be bound by the terms and conditions of this Creative Commons Attribution-NonCommercial 4.0 International Public License ("Public License"). To the extent this Public License may be interpreted as a contract, You are granted the Licensed Rights in consideration of Your acceptance of these terms and conditions, and the Licensor grants You such rights in consideration of benefits the Licensor receives from making the Licensed Material available under these terms and conditions.

Section 1 – Definitions.

Adapted Material means material subject to Copyright and Similar Rights that is derived from or based upon the Licensed Material and in which the Licensed Material is translated, altered, arranged, transformed, or otherwise modified in a manner requiring permission under the Copyright and Similar Rights held by the Licensor. For purposes of this Public License, where the Licensed Material is a musical work, performance, or sound recording, Adapted Material is always produced where the Licensed Material is synched in timed relation with a moving image.
Adapter's License means the license You apply to Your Copyright and Similar Rights in Your contributions to Adapted Material in accordance with the terms and conditions of this Public License.
Copyright and Similar Rights means copyright and/or similar rights closely related to copyright including, without limitation, performance, broadcast, sound recording, and Sui Generis Database Rights, without regard to how the rights are labeled or categorized. For purposes of this Public License, the rights specified in Section 2(b)(1)-(2) are not Copyright and Similar Rights.
Effective Technological Measures means those measures that, in the absence of proper authority, may not be circumvented under laws fulfilling obligations under Article 11 of the WIPO Copyright Treaty adopted on December 20, 1996, and/or similar international agreements.
Exceptions and Limitations means fair use, fair dealing, and/or any other exception or limitation to Copyright and Similar Rights that applies to Your use of the Licensed Material.
Licensed Material means the artistic or literary work, database, or other material to which the Licensor applied this Public License.
Licensed Rights means the rights granted to You subject to the terms and conditions of this Public License, which are limited to all Copyright and Similar Rights that apply to Your use of the Licensed Material and that the Licensor has authority to license.
Licensor means the individual(s) or entity(ies) granting rights under this Public License.
NonCommercial means not primarily intended for or directed towards commercial advantage or monetary compensation. For purposes of this Public License, the exchange of the Licensed Material for other material subject to Copyright and Similar Rights by digital file-sharing or similar means is NonCommercial provided there is no payment of monetary compensation in connection with the exchange.
Share means to provide material to the public by any means or process that requires permission under the Licensed Rights, such as reproduction, public display, public performance, distribution, dissemination, communication, or importation, and to make material available to the public including in ways that members of the public may access the material from a place and at a time individually chosen by them.
Sui Generis Database Rights means rights other than copyright resulting from Directive 96/9/EC of the European Parliament and of the Council of 11 March 1996 on the legal protection of databases, as amended and/or succeeded, as well as other essentially equivalent rights anywhere in the world.
You means the individual or entity exercising the Licensed Rights under this Public License. Your has a corresponding meaning.
Section 2 – Scope.

License grant.
Subject to the terms and conditions of this Public License, the Licensor hereby grants You a worldwide, royalty-free, non-sublicensable, non-exclusive, irrevocable license to exercise the Licensed Rights in the Licensed Material to:
reproduce and Share the Licensed Material, in whole or in part, for NonCommercial purposes only; and
produce, reproduce, and Share Adapted Material for NonCommercial purposes only.
Exceptions and Limitations. For the avoidance of doubt, where Exceptions and Limitations apply to Your use, this Public License does not apply, and You do not need to comply with its terms and conditions.
Term. The term of this Public License is specified in Section 6(a).
Media and formats; technical modifications allowed. The Licensor authorizes You to exercise the Licensed Rights in all media and formats whether now known or hereafter created, and to make technical modifications necessary to do so. The Licensor waives and/or agrees not to assert any right or authority to forbid You from making technical modifications necessary to exercise the Licensed Rights, including technical modifications necessary to circumvent Effective Technological Measures. For purposes of this Public License, simply making modifications authorized by this Section 2(a)(4) never produces Adapted Material.
Downstream recipients.
Offer from the Licensor – Licensed Material. Every recipient of the Licensed Material automatically receives an offer from the Licensor to exercise the Licensed Rights under the terms and conditions of this Public License.
No downstream restrictions. You may not offer or impose any additional or different terms or conditions on, or apply any Effective Technological Measures to, the Licensed Material if doing so restricts exercise of the Licensed Rights by any recipient of the Licensed Material.
No endorsement. Nothing in this Public License constitutes or may be construed as permission to assert or imply that You are, or that Your use of the Licensed Material is, connected with, or sponsored, endorsed, or granted official status by, the Licensor or others designated to receive attribution as provided in Section 3(a)(1)(A)(i).
Other rights.

Moral rights, such as the right of integrity, are not licensed under this Public License, nor are publicity, privacy, and/or other similar personality rights; however, to the extent possible, the Licensor waives and/or agrees not to assert any such rights held by the Licensor to the limited extent necessary to allow You to exercise the Licensed Rights, but not otherwise.
Patent and trademark rights are not licensed under this Public License.
To the extent possible, the Licensor waives any right to collect royalties from You for the exercise of the Licensed Rights, whether directly or through a collecting society under any voluntary or waivable statutory or compulsory licensing scheme. In all other cases the Licensor expressly reserves any right to collect such royalties, including when the Licensed Material is used other than for NonCommercial purposes.
Section 3 – License Conditions.

Your exercise of the Licensed Rights is expressly made subject to the following conditions.

Attribution.

If You Share the Licensed Material (including in modified form), You must:

retain the following if it is supplied by the Licensor with the Licensed Material:
identification of the creator(s) of the Licensed Material and any others designated to receive attribution, in any reasonable manner requested by the Licensor (including by pseudonym if designated);
a copyright notice;
a notice that refers to this Public License;
a notice that refers to the disclaimer of warranties;
a URI or hyperlink to the Licensed Material to the extent reasonably practicable;
indicate if You modified the Licensed Material and retain an indication of any previous modifications; and
indicate the Licensed Material is licensed under this Public License, and include the text of, or the URI or hyperlink to, this Public License.
You may satisfy the conditions in Section 3(a)(1) in any reasonable manner based on the medium, means, and context in which You Share the Licensed Material. For example, it may be reasonable to satisfy the conditions by providing a URI or hyperlink to a resource that includes the required information.
If requested by the Licensor, You must remove any of the information required by Section 3(a)(1)(A) to the extent reasonably practicable.
If You Share Adapted Material You produce, the Adapter's License You apply must not prevent recipients of the Adapted Material from complying with this Public License.
Section 4 – Sui Generis Database Rights.

Where the Licensed Rights include Sui Generis Database Rights that apply to Your use of the Licensed Material:

for the avoidance of doubt, Section 2(a)(1) grants You the right to extract, reuse, reproduce, and Share all or a substantial portion of the contents of the database for NonCommercial purposes only;
if You include all or a substantial portion of the database contents in a database in which You have Sui Generis Database Rights, then the database in which You have Sui Generis Database Rights (but not its individual contents) is Adapted Material; and
You must comply with the conditions in Section 3(a) if You Share all or a substantial portion of the contents of the database.
For the avoidance of doubt, this Section 4 supplements and does not replace Your obligations under this Public License where the Licensed Rights include other Copyright and Similar Rights.
Section 5 – Disclaimer of Warranties and Limitation of Liability.

Unless otherwise separately undertaken by the Licensor, to the extent possible, the Licensor offers the Licensed Material as-is and as-available, and makes no representations or warranties of any kind concerning the Licensed Material, whether express, implied, statutory, or other. This includes, without limitation, warranties of title, merchantability, fitness for a particular purpose, non-infringement, absence of latent or other defects, accuracy, or the presence or absence of errors, whether or not known or discoverable. Where disclaimers of warranties are not allowed in full or in part, this disclaimer may not apply to You.
To the extent possible, in no event will the Licensor be liable to You on any legal theory (including, without limitation, negligence) or otherwise for any direct, special, indirect, incidental, consequential, punitive, exemplary, or other losses, costs, expenses, or damages arising out of this Public License or use of the Licensed Material, even if the Licensor has been advised of the possibility of such losses, costs, expenses, or damages. Where a limitation of liability is not allowed in full or in part, this limitation may not apply to You.
The disclaimer of warranties and limitation of liability provided above shall be interpreted in a manner that, to the extent possible, most closely approximates an absolute disclaimer and waiver of all liability.
Section 6 – Term and Termination.

This Public License applies for the term of the Copyright and Similar Rights licensed here. However, if You fail to comply with this Public License, then Your rights under this Public License terminate automatically.
Where Your right to use the Licensed Material has terminated under Section 6(a), it reinstates:

automatically as of the date the violation is cured, provided it is cured within 30 days of Your discovery of the violation; or
upon express reinstatement by the Licensor.
For the avoidance of doubt, this Section 6(b) does not affect any right the Licensor may have to seek remedies for Your violations of this Public License.
For the avoidance of doubt, the Licensor may also offer the Licensed Material under separate terms or conditions or stop distributing the Licensed Material at any time; however, doing so will not terminate this Public License.
Sections 1, 5, 6, 7, and 8 survive termination of this Public License.
Section 7 – Other Terms and Conditions.

The Licensor shall not be bound by any additional or different terms or conditions communicated by You unless expressly agreed.
Any arrangements, understandings, or agreements regarding the Licensed Material not stated herein are separate from and independent of the terms and conditions of this Public License.
Section 8 – Interpretation.

For the avoidance of doubt, this Public License does not, and shall not be interpreted to, reduce, limit, restrict, or impose conditions on any use of the Licensed Material that could lawfully be made without permission under this Public License.
To the extent possible, if any provision of this Public License is deemed unenforceable, it shall be automatically reformed to the minimum extent necessary to make it enforceable. If the provision cannot be reformed, it shall be severed from this Public License without affecting the enforceability of the remaining terms and conditions.
No term or condition of this Public License will be waived and no failure to comply consented to unless expressly agreed to by the Licensor.
Nothing in this Public License constitutes or may be interpreted as a limitation upon, or waiver of, any privileges and immunities that apply to the Licensor or You, including from the legal processes of any jurisdiction or authority.


The SOVERSION/library version is in general not the same as the package version, is it coincidence that they are quite similar for CheMPS2?

See also https://www.gnu.org/software/libtool/manual/html_node/Updating-version-info.html


#10 ale źle jest


Looks good, thanks.


А гэтыя метады чаго не зрабілі?


It was a copy/paste mistake when pretty printing a JS object that was already in JSON format. I'll fix the docs.


Until now, I've just been fixing bugs and adding features to it as I encountered them while working on unrpyc, so I didn't bother to alter the readme. By now I'm pretty sure it'll never get accepted as a pull request though, so I'll probably just start using it as a separate project and edit the readme to be a bit more accurate.

Thanks for the advice.


Closes #215


No, this should be put back to `1`.


I do not agree with this restructuring.
The distro repositories are so that anyone could check them out and get going. The new structure you propose sets this objective a bit more difficult (now users have to go to sub folders and ignore, or copy other parts of the repository), plus it adds a new meaning to de repository (now it holds also generic folders, the generic minimal configuration folder).

observe Runner-Folder/runner/configurations already contained a most recent and complete configuration folder, thus rendering the new folder redundant.


@LOCOSP, czyli każdy inny użytkownik będzie musiał to zainstalować dokładnie w tym samym katalogu co Ty. Usuń bezwzględne ścieżki i zostaw samo `lirc/.....` bo przecież `lircd.conf` jest w `config/` a konfigi poszczególnych urządzeń są w `config/lirc/....` więc zrób ścieżkę względną.

~~~~
include "lirc/panasonic-th-42pv80p.conf";
include "lirc/technics-sa-ex310.conf"
~~~~

a w komentarzu commita wpisz `fixes #10`


Then how should I best deal with comment #5 from
http://lists.alioth.debian.org/pipermail/debichem-devel/2015-June/005954.html
?

2015-06-14 13:49 GMT-04:00 wpoely86 <notifications@github.com>:

> No, this should be put back to 1.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SebWouters/CheMPS2/commit/2468b307b9cabd91a558d83bd651dc27982dd522#commitcomment-11670721>
> .
>

--
****************************************************

  dr. ir. Sebastian Wouters
  Fellow of the Belgian American Educational Foundation
  Princeton University
  (address) Department of Chemistry
            Frick Laboratory 351
            Princeton, NJ 08544, USA
  (e-mail)  sebastianwouters@gmail.com

****************************************************


indeed they are different. Do you understand where the 102mmHg (=0.13bar) come from? Regarding int vs. double: The old deco code is in part based on some imported legacy code from HW. They use bars. Of course, in the deco calculation at some point one needs to do floating point (e.g. in the tissue loadings) and one should not need to convert too often. I admit, the current code is a bit of a mess in this respect, but the general rule should be to use int as long as it makes sense and then switch to floating point (as I said without converting too often).


As far as I know, the only place quotes need to be escaped is inside a typemap.  At least this has worked correctly in all of my tests.


Doing this in perl requires a symbolic ref which requires `no strict 'refs'`.  I don't really like having to do that, so I'm leaning towards leaving this in XS.


w tym commicie wykonałeś akcje z kilku ticketów, i robi się młyn, zeby to analizować. Rób ticketami i nic więcej. Dla configa był osobny ticket. To tak na przyszłość.

 #11


Maybe for Videotoolbox the hwaccel in FFmpeg should output NV12 by default? That's what the hardware produces natively.


just getting to known github

still feeling around


Why use sys.version here?  sys.version_info.major provides us with an integer to compare to instead of comparing against a string.  Comparing against the integer seems like the preferred method.


That's no problem. As long as `libchemps.so.1` is present. You can also put `1.0.0` here. But only increase this version number if an ABI breakage occurs.


Should also pull in the new upgrades to these lists below, 23 for health (Personal Training), 24 for auto-fire (AFK Equipment), and 25 for damage (New Mouse Button)


Jetzt hast du aber die Änderung von PhilippK nicht gemacht


The only reasons alpm_release will fail is if there's an active transaction or the handle is NULL.  Neither should occur and even if one did there's nothing that can actually be done about it at this point.


OK, I reverted it.


`logger.log` must be called after `this.items.put`, so if any exception is thrown at runtime, the attition of the item is not logged.


This, though returning immediately, does not wait for the trade to complete.  Most real world situations require the response to show transaction status.  I'd be very interested to see the sample with a response from the consumed event.


Yes, I agree with what you say. It seems the motivation here is to support python 2.6, where sys.version_info.major isn't available.


Why don't you export your utils object under your own namespace? This can easily have collisions with other bad-programmed plugins, which is not your case. For example for TiddlyPouch I have my own namespace under $tw.TiddlyPouch and there I have my own utils collection there.


need `#if HAVE_VIDEOTOOLBOX_GL` around the struct and `#if HAVE_VDA_GL` around the other one (and their respective init functions)


Его не нужно каждый раз создавать, если при попытке считать число происходит ошибка, то программа прерывается, и те символы, которые были в потоке ввода там остаются, поэтому при повторном вызове ```nextInt()``` считываются те же символы, и происходит та же ошибка, это можно предотвратить добавив очистку буфера ввода в секции catch


Thanks!


Ok, if the intention is to support 2.6, then we could do:

if sys.version_info[0] >= 3


Das kann ich nur nachstellen wenn die RecordTimer.py nicht angepasst ist. Momentan habe ich keine Erklärung dafür. Zumal diese Stelle bei jedem Neustart oder Abschalten ausgeführt wird.
Man kann das abfangen. Aber wie gesagt,  etwas seltsam dieser Fehler.


Hey @stevenrombauts what's this about? This now fails when installing on nooku platform as obviously KoowaAutoloader doesn't exist...


Например можно добавить: ```scanner.nextLine();```

Ну а вообще в коде есть проблемы которые обсудим на занятии.


Remove?


Now is much simpler, the user just point the config file he/she wants to use.


Add so it can be used to decide wether to compile `video/out/gl_hwdec_vda.c`

```
+    } , {
+        'name': 'videotoolbox-vda-gl',
+        'desc': 'Videotoolbox or VDA with OpenGL',
+        'deps': [ 'videotoolbox-gl', 'vda-gl' ],
+        'func': check_true
+    }, {
```

unfortunately our build system is a bit ugly in this regard.


Yes, that seems reasonable.


Well, okay, but you're basically doing the same thing (converting a string
package name to stash) in XS which is harder to read.

On Sun, Jun 14, 2015 at 1:53 PM, andrewgregory <notifications@github.com>
wrote:

> Doing this in perl requires a symbolic ref which requires no strict 'refs'.
> I don't really like having to do that, so I'm leaning towards leaving this
> in XS.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/juster/perl-alpm/commit/156d206e4b0d05184506715dbf00e69e3fbee641#commitcomment-11670732>
> .
>


Oh yeah that's right. Sorry I was going too fast.


Why `-?` for `Infinity` has been removed?


total should be total for the specific query (which is not used now), or if not used the where from query should not be used?


Yeah, but it's for consistency's sake. No idea why you'd want 6 consecutive empty in a file.


Right, 1.0.0 would also be ok, though if you don't plan to track non-breaking changes, probably not bothering is better, sorry for the confusion.


This, like the comment I made at reactor-quickstarter, does not return the thumbnailing status.  please update the sample to do a transactional view of the whole process, or better yet, to return the thumbnail image upon completion, instead of using a separate GET request.


Sorry @felixhayashi . I just realized that I misunderstood the export call. This is because my lack of experience with the "node-like" mechanism. This is only exported to the modules that requires it. Is it a good idea if I reuse your module for uuid generation on my deserializer module? just by doing

var utils =require($:/plugins/felixhayashi/tiddlymap/utils.js)


Can someone explain in layman's terms what this warning message is? My understanding so far is that when someone opens a pedigree there is a box to warn them that there may be sensitive information.


I don't think you need sv_2mortal here. It might actually do bad things.


apt-get update is longer necessary since https://github.com/mail-in-a-box/mailinabox/commit/0754ce01b16f88b14e29d4d5acdd62bdf902c11e.


Should verify the tag before checking it out.


Added some spaces and cleaned up some, as the confusing scopes are... well... confusing.


why two casts here?


ыы


Done


Not sure if this is required. At least on Nexus 9 stock 5.0.2, the libart-compiler.so file doesn't exist in lib64. I think dex2oat is the only binary that needs it, and it's compiled as 32-bit binary for me.


pagma?


Sorry about that, my brain was in jQuery.each mode again just like in #99.


I think it's just another constant, It represents other gases in the tissue compartment, always present, never changes, has influence on crushing pressure calculation (sums with N2 and He to total tissue tension).
I will keep ints as long as possible, no problem.


resolves #367


resolves #368


Nice!


Wygląda OK


Hell yeah. Thats cool.


@DomT4 Thank you kindly!


I would change this part to:
```javascript
 walk(dom, function(el) {
    if (el._tag && !el._tag.mounted) el._tag.mount()
 })
```


Working on fix #110


Please update it !


From perlapi: "sv_setsv Copies the contents of the source SV "ssv" into the destination SV "dsv".  The source SV may be destroyed if it is mortal, so don't use this function if the source SV needs to be reused."  The source SV is copied, not used directly so it's marked as mortal so it doesn't just stick around forever.


I would rename the `inStub` helper to `isInStub`


Moved to here since we know the mount path starting here and hence can also unmount the image in the trap


From what I understood, this is really the only line that makes this config different. Since we just want to override things that are already in the default config, that's all we should need here.


Log to syslog & stdout (which gets redirected to /var/log/webcamd/webcamDaemon.log)


Can you please remove these comments or expand on them to explain their meaning?


Looks great: good job. Let's have our meeting and I will accept it right after!


Yeah, I think that would be better!


Ends are another one where having binding would be cool. This would allow you to write things like codensity more compactly:

```
End(X. Power(CTT(A; F(X)); F(X)))
```


I was wondering -- this class seems to copied from somewhere else. Do you know under which license was the original released? Just want to make sure we are ok from the legal point of view.


This might be interesting to also display in OctoPrint somewhere alongside the octopi version via a plugin (no problem quickly whipping this up ;)) so we can better track what users are actually running when they encounter bugs.


Something like this would be good
![black and white](https://cloud.githubusercontent.com/assets/608650/8149819/015d9f04-12a0-11e5-945a-3832d383a129.jpg)

Right now looks like this (I had to use a software to paint the lines in black)
![black and white](https://cloud.githubusercontent.com/assets/608650/8149857/4de32f50-12a1-11e5-9255-b324e5a5d33a.jpg)


Isn't it meant to be `console.trace === fn` at the end?


We really should always die when we encounter an error... for commands that may fail there's always the `|| true` construct.


`\` is a legal character in Linux filenames.  Splitting on it is incorrect.


This allows you to check your configs -- `CONFIG_ONLY=yes ./build` and you only get the printout and nothing else, very helpful for debugging.


thank you @ezyang can you also manually trigger a change at packagist?

> Existing packages without auto-updating (GitHub/BitBucket hook) will be crawled once a week for updates. When a hook is enabled packages are crawled whenever you push, or at least once a month in case the crawl failed. You can also trigger a manual update on your package page if you are logged-in as a maintainer.


Fail early and hard...


Ah, yes, they're pydev (and IDE) specific annotations that slipped through.

How do you want me to deal with your requested changes - add more commits to this branch, or rework and re-push the branch (i think the comments might then get lost).


I moved the "generic build tools" out of the individual component build since they might be needed by other components (e.g. mjpgstreamer and haproxy both want cmake, so it makes sense to move this up here)


I noticed the OctoPrint install failing sometimes, hence the custom timeout... not sure if that was only a local problem though. Default value is 10 or 15 I think.


Thanks for looking into it! I'll keep an eye on that issue. Let me know if any more info would help.


The KVO notifications are provided for free if you set a property. If you use instance variables, you have to fire these yourself. When I wrote this code, I didn't understand that so I just copied what other code samples did (they used ivars), which would cause duplicate KVO notifications.

—
Sent from Mailbox

On Sun, Jun 14, 2015 at 8:56 AM, jklimaat <notifications@github.com>
wrote:

> What is the reason for removing KVO here and in the other locations? Per Apple's documentation, custom implementations of NSOperation must generate KVO notifications when they implement the start method.
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/skunkworks/TCCMapTileAnimation/commit/67d06be74d40000156ddcf0b9fb4ead80d85b4f0#commitcomment-11670366


wait, I just realized why this was correct before you changed it. you didn't use the updated code in the iterators!!!


I moved off the deprecated setting some time ago on the `next` branch (https://github.com/wincent/corpus/commit/28dd982dc7aeeb0b9c9ebf07263f54c8c3051290), and not by disabling it, which seems like the wrong thing to do.

Sorry about the lack of visibility of the `next` branch. It is mentioned in `HACKING.md`, but, er, only on the `next` branch so far... (https://github.com/wincent/corpus/commit/3eee97294040798a8531835d8b50c3fb384b5c64).


@JuanitoFatas

Including
```
- fix wrong comma
- remove extra spaces
```
in the commit message doesn't make sense; in the history of master there is no wrong comma or extra space.


Saved = Solved *


I don't see the errors you're mentioning in your commit message, btw...


Thanks!


I am not sure but think this cause 0 problem : objList.Add((int)0);


This is a valid issue with session handling. Right now the `session_save_path` is set to the default of  `/var/lib/php/session`, and that whole tree doesn't exist in precip. Not super sure why it hasn't come up yet, possibly because Drupal doesn't save to the default session path, and instead saves to Drupal's temp path, which we're manually setting as `/tmp`?

Please put in a PR and I'll merge it.


FYI, I'm using NPM's `shrinkwrap` feature, which means that `npm install` effectively ignores this file. If you want to update something for reals, you have to run `npm shrinkwrap` after updating. TBH, I am not even sure why I have this in the `package.json`; I would have expected it to be pulled in indirectly via my use of `babel-eslint`.


typo!


This looks like it gets us part of the way there; like I said in my comment on the PR, we want selected note deletion to work properly when the OmniBar is focused too.


Thanks.  I have been with the family and not had a chance to look at it yet.

Is it the combination of the closure and anonymous struct that causes the file to bro autogenerated?

That seems very odd given that the compiler should have all the information to emit symbols for this.  Would it be wise to test that this condition actually (continues) to behave in this manner?

Seems like an edge case that the go team might fix in the future.


Cheers!


german layout based on http://www.zav.cz/german/compkomasch.htm


@chelmuth, @nfeske: please add to staging


I don't understand the benefit of `inline` here. Did it really help performance? I'm looking at this file for https://github.com/fsprojects/Paket/issues/874


scrdest iis rouge! :red_circle:


why do we want to implement these two functions?


Why is this function changed?


Use t.Log and the output will only show up with -v


great idea!  It's nice how Dropbox works like that.

Haven't done any resume updates in a while, but when ERAS comes around I
will.

You're much up to date on applying to scholarships and all.  I should learn
from you :)

On Sun, Jun 14, 2015 at 1:41 PM, Erik Reinertsen <notifications@github.com>
wrote:

> Instead of hosting my CV on my site, I keep the .pages and .pdf in my
> Dropbox public folder and link to the .pdf on my website. This means
> whenever I revise the CV, it automatically syncs with Dropbox, but the URL
> remains the same. So my CV is always up to date without the additional step
> of uploading the revised version to Github pages.
>
> You can achieve a similar but slightly less automated workflow by keeping
> the .pages and .pdf in your repo so the latest version is always pushed
> when you push rest of changes.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/erikreinertsen/jgmalcolm.github.io/commit/ae355719eb1a5234641ff54bdb97cd9138754019#commitcomment-11670698>
> .
>


It's usually [good practice](http://stackoverflow.com/questions/729692/why-should-files-end-with-a-newline) to keep the newline at the end of a file.


Could we remove the @author annotations completely?

Having both the annotations and the license(copyright) is conflicting.
https://github.com/SmBe19/SG002/blob/master/LICENSE


I used the gmls22 modification and it started to work with owncloud 8.0.4 and the cloned repo. Before that, the users section of the OTP was a blank page and the OTP area in the Personal settings wont generate QR-Code nor seed.


Bound to the context of the EventEmitter instance in EventEmitter.emit(): https://github.com/joyent/node/blob/master/lib/events.js#L106


We have been incorrectly putting multiple elements inside components that are really quite independent of each other (though thematically similar).  Each component should really only have elements which are functionally inseparable from each other.


@esoergel you on it?


Hola @danielo515

> This is only exported to the modules that requires it.

correcto!

But I would not require it like this. Maybe it changes its import title. Better access it like `$tw.tmap.utils`. But be warned, maybe some functions will change names or the function signature changes. I try not to do it but sometimes it is necessary. What functions in utils are you interested in? Maybe just copy the functions you need and use them for your own utils module?

> $tw.TiddlyPouch

Also I would suggest you have a short form of tiddlypouch. It is annoying to have long namespaces. I started with `$tw.tiddlymap` and ended up using `$tw.tmap`.


I'm starting to think this issue has to do with Raspbian targeting ARMv6.


It don't think it has to do with the closure. The "<autogenerated>" is added because we are calling `.Close()` but we haven't embedded a `io.Closer`, so we are actually calling `c.Close()` on `nil`.

I believe this test would be enough. It should be OK for them changing the internals as long as our tests still pass. :)


Should we consider making StatisticalModel.mean a val instead of a def ? This would avoid this kind of problem for users that might think (as I did) that the mean is computed once and warped at every call.


For quick testing hands like a Royal flush, i could write any array within the brackets. Once i was done testing i forgot to remove the brackets and leave only "pHand"


didn't update version number 8-)


Do we still need `async_storage.js` now that we have removed the preferences stuff?


Debug leftover.


What?


`map CATS::SourceManager::load($_->{guid}, $self->{modulesdir}), @$sources;`


Pedro and I are in active conversation about licensing. This is not the final form of the file, and it will be mostly rewritten. But yes, I plan to change the copyright to the original and apply them in different form.

( Btw, here is an other example to add own copyright before the original: https://github.com/opntr/opBSD/blob/op/hardened/current/master/sys/amd64/amd64/pmap.c )


Without this, an ECL [bug](https://gitlab.com/embeddable-common-lisp/ecl/issues/76) is triggered.


@vcunat: Thanks. (I was hoping upstream would make a new release. I hate it when distros have to do the work of upstream.)


Could we remove this as it is commented out?


Nit: remove `type="text/javascript"` as that is default


Not sure.. I don't know github all that well ;).  Whatever you can do to clean-up the couple issues -- I don't think preserving the comments is that important.


Nit: replace `/>` by `>` to not avoid XHTML stuff. Also one line below this one.


Nit: this line should also have `type="text/css"` for consistency with the line above.


1) Use `qr`
2) Use `&& !` instead of `and not` inside of expressions.
3) Why is the second regexp needed? Did you convert `$guid` to regexp incorrectly?


Please be more careful when committing directly.


@myrdd can this be merged to `dev-1.0` or is there a reason not to merge? Very useful.


Now that the PDFHistory code has been removed, can't we remove this too?


Do you expect zero `id`  or `stype`? If not, this condition is redundant.


Actually, the way it was done was OK in that regard -- but numbering (versus labelling) is still cumbersome.


<b>waddup dog<b>


Nit: PDFs without quote


nun mush pimp


Ok pour remplacer la page déjà ouverte au lieu d'en ouvrir une différente à chaque fois, mais on en discutera au Scrum.


Would this be better? `// Ghostscript sometimes returns 'Untitled'. Prevent setting the title to 'Untitled`.


@myselfjivan: You don't want to fix this ?


Why do you check for `isNan` here? As far as I can tell, `level` is always a value and otherwise we should force that. Also, why are we updating the bar if the percentage is not a number?


@aldanor I finally made it compile & sucessfully run tests on Win7. I always got a corrupt stack when calling the lazy initialized symbols, only recompiling hdf5 with gcc helped at the end (I tried several configs with different calling conventions without success, the original binaries from HDF group are compiled with MSVC). I can send you a PR, just want to make sure that all my trials are cleaned-up beforehead.


Nit: use capital w.


break statement ?


Why is this? Does the old redux stick around indefinitely and just the dispatcher get replaced?


Looks like React's auto-binding is at work:

![screen shot 2015-06-14 at 12 15 54 pm](https://cloud.githubusercontent.com/assets/169760/8150057/4c85fe38-128f-11e5-8dcf-01296c33e976.png)


Can we use the `DEFAULT_SCALE_VALUE` constant here?


Nit: unnecessary newline


I also hate this. The best thing is that the change merged upstream didn't actually work...


Oops, #27


This code is wrong. Beside that, the simplest code which would have made the test pass would have been:

```
bool isEmpty() const {
    return true;
}
```


Nit: PDF.js :)


Yeah, that's [exactly what happens](https://github.com/gaearon/redux/blob/245c92096b2eb7e5474f23a794d3cf23bf990cb4/src/components/createProvider.js#L29-L37) :-)

Easier to replace the dispatcher than subscriptions/state. (Although either approach could work.)


Getting error on iMac:
fatal error: 'CL/cl.hpp' file not found
Is it having problems picking up the accelerate framework?
http://pastebin.com/u0uB1X6P


Can we add a new line above `translate` just like we do for all other objects, for readability?


This is missing for sure!
Created: https://github.com/killbill/killbill/issues/335


I accidentally pushed this straight to master instead of my own branch.


Move this to the top of the file if possible.


Nit: remove extra space before colon at `url`: `{url: url, ...}`


We should talk. The caser i 'optimized' by bailing out early is for when plugin is useless to make progress in the resolution of that case (being a PENDING or UNKNOWN) case.


Well, the problem is that every val occupies memory. In this particular case, it might still make sense, because the mean would arguably be requested more that once. I vote for "lazy val" :-)


If we do not run the locale target anymore, aren't the files from https://github.com/yurydelendik/pdf.js/commit/fff086645b7a3fa5daeefe6e44f9ac5b5ab5abad#diff-5dba6a2f9a0235d9e5b2b75ef5a6f774R32 missing then?


 Хорошо, спасибо

Воскресенье, 14 июня 2015, 11:03 -07:00 от Igor Nesterenya <notifications@github.com>:
>Например можно добавить: scanner.nextLine();
>Ну а вообще в коде есть проблемы которые обсудим на занятии.
>—
>Reply to this email directly or  view it on GitHub .


@Optimusprimeums I just remembered that I updated FFmpeg for OS X and you said there might be problems with it, is that with the version we just released? I updated it from the same source that we used in the previous build BTW


COLONS

BAD NO NO COLONS BAD BAD BAD


Instead of using 1, 2.
Declare 1, 2 as enum or Constant with a proper name.
And you can replace switch case with a single if statement.


@vido89 Don't upgrade to Ogre 2.0 yet, the upgrade is not finished and it doesn't even compile on Linux. Just wait until the Ogre 2.0 branch gets merged into master.


Nice.


A space before ';'


lk


Does it make sense to merge with an empty object? Can't we just use `DEFINES` instead?


Hey @voat some of these pull requests have already been done in the API code, like this one for example. Since few contributed before this wasn't an issue but now it is.

@johnduhart if you could coordinate with me before you spend time on refactoring, I will be able to tell you areas that are a OK to work on and areas that will be replaced when new code is introduced (like this PR will be unfortunately).


corrected image paths


Don't copy the script in home folder.
I did it for my simplicity because I didn't set the pre-start script.


`getXhr` has become obsolete now. Can we move it and change `getXhr: getXhr` below to `getXhr: new XMLHttpRequest()` instead?


Yea sure take your time, thanks for trying it out. I wonder why doesn't it work with MSVC build, is there a way to figure it out?


ΤΛ
2011020


Same here, can we remove this `NetworkManager_getXhr()` function and just put a `new XMLHttpRequest()` here instead?


ΔΑΜΑΛΗ ΔΗΜΗΤΡΑ


This causes a conflict when building for another machine. This had been reported to me by a guy on yocto ML. I'll reveret this and add [my solution from meta-amlogic](https://github.com/linux-meson/meta-amlogic/commit/c5e4255af6c0026223c1d44bcb824ea6f607124c)


Can we fix the small typo `userAgant` -> `userAgent` below too?


@gizmo98 this change no longer worked - the file moved, but also changed too - do you know if there is still an issue this is needed for and if so, could we feed something upstream ? No rush - just letting you know. Cheers.


could probably run at 1/10th but i might just rewrite it to use a dijkstra map instead :^)


@jackpoz fixed a minute after this commit, so it couldn't cause any issue ;-)


Sorry @GrahamCampbell  I didn't found it. (thanks for StyleCI it's look pretty amazing :+1: )


Can you comment on the function of L864 and L865? Using the stopAutoUpgradeManager() results in a 'TryUpgrade_upgradeManager is not defined' error currently.


you should add something in Catch


Toi palstotus toimii tällä hetkellä vain koosteessa.


тут m ни при чем. 3 - это длина check, а не b
может, разумно добавить константу k=3, рядом с определением check.

хотя когда таких констант больше двух, их лучше переименовать более вменяемо, например na, nb и ncheck


https://rawgit.com/avoimet-oppimateriaalit-ry/avoin-taulukkokirja/master/matematiikka/kooste.html
Ohjeet miten kooste päivitetään löytyy README-tiedostossa.


Good fix. Completely forgot about add-on.d


It is an error "if" the end of dst is within src.


scottys header fn only returns one header, no need for find here.


Did you mean to rename the `args.output_directory` to `args.output_dir` below?


hm, I did not do this manually... nuget magic?


How about

   $[autologin} && configure_displaymanager_autologin

:)


Thanks for your feedback, Guido!  It's very helpful.  After trying a lot of different approaches, I've decided to go with a directory-per-chapter approach.  I spent a lot of time thinking about repository maintenance and community contribution and, though I agree with you that using Git as you would on a real project is compelling, it creates a maintenance issue that I just don't feel I'll have time to deal with (and isn't something I can delegate since it would involve changes to the book copy as well).  Working on that this weekend!


It's crafted with rubber boots and an aqua middle gem. Also needs a new texture


What is it for ?


s/Geoclue/Geoclue2/ ?


I assume it's not really needed, but I always try to inline small functions away. Is it a problem?


They haven't split/updated the distribution for rpi2 vs rpi1?


Thanks for pointing that out! :+1:


i can check this later and clean the build and update for multimachine build, all wetek changes in git was clean


Definitely do this on a branch next time. All of these projects are continuously integrated so the master branch should be considered "hot".


Could you keep compatibility with php 5.3?


Thanks for the format fix that broke the tests. You merged faster than I could fix it. :-)


Did you get it to boot? Because I didn't..

Here is the logcat
http://pastebin.com/diZqNEVL


Yeah, similar problem here https://drdump.com/DumpGroup.aspx?DumpGroupID=308660 also caused by evga crap. We can add safe checks, they won't harm and add "response link" on DrDump to indicate that EVGA software is responsible for the crashes...


Shouldn't this method be named ``setState_RP`` for parallelism with ``setState_HP`` etc. as below?


I think the error checking should be done at the lowest level implementation of ``setPressure``, e.g. as is done for ``Phase::setDensity`` and ``Phase::setTemperature``. This way we take care of all the interfaces in one place.


so what's wrong with this??... if you have found a type...pls show...
if not...
then ... yes im sure thats needed
or do u want me to upload a avi to proof it.


These checks are redundant if checking is done at either a higher or lower level.


Let's add where these come from so no one else google's `javascript fastest way to flatten object` like I just did and finds the same stackoverflow solution ;-)
http://stackoverflow.com/questions/19098797/fastest-way-to-flatten-un-flatten-nested-json-objects


Not used?


NP :) And yeah, trying to update all our logging to have levels too.


Thats actually so much clearer - that's why I left myself such a long comment.  I especially like how the lines read naturally.


Vasan


It's probably best to keep it in a separate branch of tupac_test -- tupac test, as a repo, should be setup such that anyone new to the office, or any of us if we start a new feature, can come in, & branch from the Main_Master, to work on something completely independent, using our libraries -- so if there is a new addition to icarus, I think it's probably best to make a tutorial, or sample branch, to show the code for it, and keep the code local to the tupac_test branch. thoughts?


s/are close/close


They will provide loaner robots, Intel Edison boards and more for you to hack on.  In Go, of course.


I can do the same fix for Mundungu as well.


Somewhere in here we're going to do some Congo hacking --- Help Erik and Brian maintain their sanity by building an integrated Conference Management platform.... etc.


looks like there is an extra "btn" here


You should say which member: `"Added ${patientId} as a member"`


Great!


Fixed, I have no idea how I messed that up. I clearly cannot copy pasta good.


新Ver製作時はその辺も考慮します。ご指摘ありがとうございます。
2015/06/14 7:32 "173210" <notifications@github.com>:

> 何ならせめてでもstatic修飾子つけませう。
>
> (GCCばっかり使っている私からすると、static修飾子をつけないとインライン展開されないのでアレな気がするのですが、VCはリンク時最適化が標準で有効だから別にいいんですかね。)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/YSRKEN/aviutl-waifu2x-cpu/commit/f0e4ddbde9e98c37069bdac2bdd8b79f12a12271#commitcomment-11668117>
> .
>


@NeilMartin Nah, Time.time isn't raw. It will have advanced by the timeScaled deltaTime already, so this code is correct.

There's a separate property called Time.unscaledTime that's unaffected by Time.timeScale.


done


added


do we need a different jquery here than we have in the static folder?


you read my mind! :O)


rax retry tests


+1 You're awesome.


If there's no data to display, the header will still show up.

And change the title, it sounds like you're signaling a problem to the user.


Ah, in that case, `envelope` is probably the correct term (matching the code)


Escape things that you put in the URL: `$xwiki.getURL('Family.CreateFamily', 'view', "patient_id=${escapetool.url($doc.getName())}")


`escapetool.url` here too.


can you push it to https://github.com/3drobotics/mavlink-solo?


Dependendo do computador, em vez de "FaceRecognition" coloque o nome da pasta onde estão localizadas as fotos e arquivos.


Since the output will be similar, and will have the same effect ( consider that those functions are never called), I`ll not re-generate the mavlink files. Feel free to do it tho.


It looks like this whole C2000 branch needs to be merged into that repo.


gt was throwing me, but I see now it is  0 or 1, not a decimal, and in this case should be 1, nice lesson, thank you!


![untitled](https://cloud.githubusercontent.com/assets/7658554/8146855/555ac2ba-127a-11e5-9253-7f8bc30d9318.png)

The icon is still not found


Hola este Commit, lo hizo Duvan Gañan, pero no cambio el usuario, para que se tenga en cuenta.

Sds


@watsonbox Hi Howard. This might be a more visible way of reaching you and letting you know I've posted a reply 2 weeks ago and am waiting on you before I can proceed with the fix:

https://github.com/Temptin/SkypeExport/issues/4

Since this is a *random* corruption in your database (as is so common with Skype), there's nothing I can do to test this on my end, so I am relying on you to help out a bit here. ;-)


@caiolima porque vocês não adicionaram os arquivos de configuração do eclipse no gitignore?
E façam um README que preste :angry:


Does not matter guys. File_exists execution time is this:

![qa2ok](https://cloud.githubusercontent.com/assets/4078041/8146860/fe5abab6-1282-11e5-9b38-712979f6da9c.png)

I'll be subtle. We're in beta for pete's sake. PF need core developer to NOT hammer fixes. Hammer issues instead. Bring the stress somewhere else... for example, Playstation. I can recommend Dragon Age Inquisition.


The more I think about it, the more I feel like we shouldn't worry about literals just yet. What makes them so weird in older versions of Julia is to the automatic expansion of block matrices, where `NULL` would need to mean something special.

Let's cycle back to literals after Julia's array literal syntax has gone through a few more cleanup passes.


JH session 21.5.2015 most corrections


I blame Github Editor.


@thanhdut DWMR. Press on.


Is that needed here?


liar.


I assumed you wanted 5.4 given that you had StyleCI configured to format the code with short array syntax.


Also, why would you want to support php 5.3 out of interest?


but if autologin is gibberish or empty its not the same


It could be bad you exposing base type for user as fully valid type.
So it could be used even as argument to CLI.
Can you expose it as separate variable?


Note that this commit doesn't work since I guess the goal was to save video options even when we change the renderer. The problem here is that you save the values to the renderer before changing it (which is not a good thing IMHO since I think a player that change a renderer and video options would expect the video options to be applied to the newly selected renderer and not on the old one).

But even without this problem, we cannot do it that way. The resolution string, for example, is renderer dependant and not compatible. For example, for D3D, it is stored like "640x480 @ 32bits" where it is "640x480" with open gl. IIRC, writting a wrong parameter did result in a crash at game start when I tried...
To make things well, we should use the renderer to know available settings and available options per settings. However, since we said we want to officially support open gl only, it is enough IMHO to restart the game without saving parameters.
Note that it's what I wanted to mean in the comments line 267 but it seems it is not clear enough. Feel free to change the comment so it is clearer if you want


Yeah, I assume the iso makers won't uncomment autologin and leave it empty.

Anyhow, even if we ask "is true" fully, its still better than double negation. ;)


Ah, so using createClass isn't the same as extending it (ES6).


Can we invert this case: if(last_char_hyphen || last_char_dot) return -ERR_INVALID_BUCKET_NAME;
Would greatly improve readability


else if


Canola?


The point is, we currently don't check the vars in profile.conf for valid values, even if we ask in full, it would not be triggered if string is gibberish or empty.
The question is, do we need a gibberish value check, or do we assume iso makers know what they are doing?


To avoid BC break, but php 5.3 is deprecated, look reasonable to break this compatibility. Could you remove php 5.3 build from travis.yml?


made it more generic, to offer up a better understanding.
remove the make.conf that i use, so as not to confuse anyone.


Please read the comments on #3008, it's explained there.


What happen when fluentd failed to stop?
Fluend binds some address/port so new process failed with resource conflict?


Seems good to me and works well


Great stuff here. I'm really pleased that you've seen that this requires multiple `it` blocks to test the different states of the validation.


Very cool!

May I ask when you plan on releasing 0.1.5, please?


haha you left "cake" in there! >< thanks... "the cake is not a lie"


Yeah, this works. `@current_user` will return `nil` until it has been set by line 22, so you can call it throughout your code.


Sorry, I see you fixed it in the last commit.


Ok, thanks! Would you know: Is it a bug that my correction in Touring view was still overwritten by the mistake in default.render?


It's best not to use "magic numbers" like this in your object design. If you wanted to use this consumption function for a different town, you'd need to rewrite it.


Looks good.


Nice!


I changed this from "only if is true, do this" to "if is anything but false, do this", because its the default behaviour we want. Double negation is just an accident of that, and the alternatives are worse.

And yes part of the reason for this is because we don't check for correct values, so at least if they are wrong, the default is sane.

Another thing, please don't comment the old code and write the new one. Git makes that completely unnecessary.


You can also use a shorter version of the `render` method for the common case of rendering a partial with local variables. `<%= render 'comments/form', topic: @topic, post: @post, comment: @post.comments.new %>`


Also: I see you removed possible value "--" I used to have, which facilitated disabling the contour line display entirely in that menu (and we did not need to translate "--"). Do we maybe want to keep this?


Well, I just commented out in case you want to revert it.

I had more a general var check for profile.conf in mind, since we added other bool vars too.


The lack of `__setitem__` was making it hard for me to test something. So I needed that, then I just thought I'd implement del while I was here for completeness.


Ok thanks.


I honestly can't remember


It's not fine everything is read at the same time in blockInputStream.process()


I introduced some duplication here—not sure how to make this call `scripts/test` and “pass through” `--watch` and whether we even care to do it.

@emmenko @acdlite


damn I missed that


I feel like you're repeating yourself here


I may have said this before but I think you could do this with one loop.


It was a problem when I was trying to extract an .fsi signature for Logging, so I removed them. I'll see if I can get the PoweShell logging working correctly in Package Manager Console and then I'll submit a pull request that will require review.


Git has his own tools for that. http://git-scm.com/docs/git-revert


"Is it a bug that my correction in Touring view was still overwritten by the mistake in default.render?"
Not sure. But you added only an option not code that draws contour lines.
"Also: I see you removed possible value "--" I used to have, which facilitated disabling the contour line display entirely in that menu (and we did not need to translate "--"). Do we maybe want to keep this?"
To disable contour lines entirely better to turn off plugin.


Apple's new OS font (San Francisco) is out so I immediately switched my CV to it: http://appleinsider.com/articles/15/06/12/apple-releases-san-francisco-system-fonts-for-ios-os-x-and-watchos-2-to-developers

Yeah I need scholarship $$$ haha.


Dear Sir,

Hope you are doing well. I was wondering if you could come up with a new update to playback of PC 9801 music files in the new foobar2000 v1.3.8. I don't think the one posted on http://www.vesta.dti.ne.jp/~tsato/soft_fb2k.html works anymore. Thank you so much.


Why are we catching here?


Yes, i think it connected with rounding (bit clamping seems was wrong i think, it mustn't affect npc movement).


See #1918


This will not work since touch values are not preserved, we really need to make an actual copy of the int value


Why are we catching here at all?


Whoops, sorry. Forgot to update the version.


I think I've found a way which indeed reverts this commit while still keeping the proper behavior when on top is used.


Do you know why the helper_plugin_.. is directly created here, instead of using plugin_load()?
I think using plugin_load let reuse the helper object again, which is fine here, isn't?


Why is this being removed?


Also this line needs to be pulled in to the main. Currently toggling the upgrade button visual


Why are we asking for IE10 here?


Not a problem, as commit history will tell, I forget it nearly every time (For thAt matter, have I even remmembered it once?....)


What's up with these random catch something?


Just curious why 99.99% instead of 100?


@oligriffiths We need that in the latest Joomla versions to make sure the framework has been bootstrapped, because Joomla invokes the composer autoloader before the plugins are loaded. Obviously this won't work on platform as KoowaAutoloader isn't there and this wasn't tested properly.

I fixed this in [46af5436](https://github.com/nooku/nooku-installer/commit/46af54364a7ba8f6325685ee915cc0f02373c57e). I see your platform components require dev-develop so it should work again for you now.


Not sure why we have a onFailure if we do not do anything in there. What can actually fail in this conversion?


Naming it soundcloud.py makes it not working on windows terminal, as it messes with the soundcloud library


Needs formatting (couple of other places in this file)


Please add new lines at the end of all files in this commit


Adding an if for safari will fail compiles for other user agents (e.g. Firefox)


indentation looks of, should be two spaces


only bluring on body will not work for most webkits.


brackets on the if line please


please do not throw styles out for gecko


@alevshunov This and 3 other remarks were fixed in the next commit. Last changes were made yesterday.


There is probably something the `my @parts = split(/.../)` regexp is missing.


Why do we need an important here?


Have you considered using [MimeKit](https://github.com/jstedfast/MimeKit) and [MailKit](https://github.com/jstedfast/MailKit) instead? MimeKit's parser was built with the intention from the ground up of handling any flavor of newline. It's also 100x faster than AENetMail's parser based on some of my preliminary benchmark findings.

This is likely due to the way the parser was designed plus all of the optimization tricks I've implemented to increase performance.

Since it doesn't do inefficient stuff like ReadLine(), but instead operates on 4k byte[] blocks at a time, it's insanely fast.


The BOM is missing the 22pF 0605 capacitors for the crystal and the antenna


Missing copyright


The compiler gives a warning if you directly cast a pointer to an integer of difference size (which is sometime useful but sometimes stupid IMO). The first cast (`int64_t`) can be omitted if we are sure that `long long` is always 64 bit long on 64 bit systems.

Actually, given the windows warning, I'm just going to use `inttypes.h` and define necessary missing macros if it is not available on Windows.


It looks like https://github.com/wincent/corpus/commit/28dd982dc7aeeb0b9c9ebf07263f54c8c3051290 has been landed on master, so this commit builds on that. Not sure why I'm seeing these errors but you are not though.

I agree that disabling this rule doesn't seem ideal, but why does it seem like the wrong thing to do? What are the alternatives?


Why are we changing this?


Gotcha. I haven't worked with `shrinkwrap` before, but now that you mention it, that would explain some of the versioning weirdness I was seeing.


First quick pass over the commit


Also mock pipe tests, filter tests, and util tests.


* FIXME: Handle the failure as it is unclear what went wrong


It is strange, why used that switch, when it is possible to use bit masks? Or it is faster?
Or maybe there is some logic in such clamping?


pushed it just in time 21:50:29 ;-)


No, but this is probably the best bet: http://sjoerd.luon.net/posts/2015/02/debian-jessie-on-rpi2/


Better not to update these since they are managed by Maven itself. You don't use configuration on these plugins AFAIK.


De nada 😉

Éric
Le 10 juin 2015 19:07, "jeromecc" <notifications@github.com> a écrit :

> Thanks for the mentoring!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/FreeMedForms/freemedforms/commit/297b23d1041c14767ced129c596cd4987019d3ca#commitcomment-11616527>
> .
>


I didn't leave any debugging statements in, but having the diff be in order is much nicer for viewing while debuggin


We might be able to get rid of this entirely with #729, but this looks good for now. Unless we want to fold that PR in here too. But maybe better to just do them one at a time to get this moving.


Always do a git pull --rebase. Merge commits are bad.
You can configure it globally by running: ````git config branch.autosetuprebase always --global````


More info: http://stevenharman.net/git-pull-with-automatic-rebase


Of course bytes(1000) means one thousand NUL bytes.


@puttitout, that's fine. These commits are meant to stop the bleeding, I don't mind if it's replaced later.

Is there a way I can get access to this new codebase?


I think that now there is no need for this member to be mutable


Version need be == 3.4.4 or >= 3.4.4 is ok?


That switch can be simplified, using some bit shift for min and max and simple if (bSigned).


Why was this changed?


@giniouxe A modal has to be open when the user click on the invite link. Have a look at how this is done on the /contacts page.


If you make the patch musl-only I'll apply it. Currently it applies for uclibc as well and I am not sure whether that removes functionality or not.


you are commenting on an old commit. the ruby things are no longer there.


no Kappa


Or https://wiki.ubuntu.com/ARM/RaspberryPi

Looks like it might be less effort to buy an odroid instead at this point.


Of my opinion, when users click "restore", they expect to see the window pop out from the hidden taskbutton.
If we don't put "raiseApplication()" here, the windows would not pop out while the application was in the "Hidden" state.


No?
https://github.com/manjaro/manjaro-tools/commit/847991443dc4661dd26d6a5c5aff090c9a0c9ebd

Can be expanded to check everything in this file.


Aww, something went wrong, with rights, i'll fix it later ;)


event.type wont compare to K_p.  event.type only has things like keydown (a key was pressed) and keyup (a key was released), if you want to pause the player ant (its animation and movement) you would need check that event.type == pygame.KEYDOWN, then check if event.key == pygame.K_p


Can we make this for all OE-A images please? It makes sense to have all EPG related activities in one place.


Thank you! This commit fixed the build. It was broken since http://ci.cgeo.org/job/c-geo/3811/


@isagalaev, may be you can remember what causes [this](f0145d6d66cf9a19883cc913c01542a41f1fd1b4) commit?


Good catch! Thanks!


Unable to merge at the moment. I will prefer to put the conditional inside the 'load_js' method. Also, Instead of using page, It will be better that we check form get_current_screen()->id == mpp_get_gallery_post_type()


Yeah, that does make a little more sense.


> Hmm could be worth it but I say we wait at least a week to make sure they are really gone.

Do we want to include IMFTrackedSample commits in that hotfix release? If not, it would make sense to revert them now (and maybe bring them back after).


Not opposed against adding this, but can you explain when there are parantheses in the ratings to clean up?


Thanks for the clue. I applied locally and now it build nice with VDA and Videotoolbox enabled :)


addons/README and po/POTFILES.in should be updated as well but this can also be done later after merging.


Thanks! This is wayyy simpler than my code. If i can, I'll merge this into the original branch and give you contributor status.


Hmm. Are we 100% sure that this isn't prone to SQL injection? It  looks like it could be. Also line 12 below.


Drop lazy initialization (because the webcam list might have changed). Don't forget to free() if not NULL.


I don't have rights, the packagist thing was made by @robo47


Νίκος


thanks for your excellent free extension , i hope my translation helps to broadcast this extension and it's great abilities .


`DebugPrint()` on this error too?


Yes of course! For example: I'm using a program that downloads all the info for my movies.
The certification comes like this: "Australia:M  / Canada:PG (British Columbia) / Ireland:15A  / Philippines:R-13  / Sweden:Btl  / UK:12A (cut) / USA:PG-13 (certificate #49491)"
As you can see, both UK & USA have parentheses.


Yeah that was my mistake, I misread what was going on. Its been fixed hahaha

On Jun 14, 2015, at 5:04 PM, DepletionSlayer <notifications@github.com> wrote:

> Why was this changed?
>
> —
> Reply to this email directly or view it on GitHub.
>


Why?


Most probably some other language was wrongly detected as Rust and I illegalized `\S` simply because I assumed that was correct based on the Rust snippet. To prevent such errors we should be much more strict in adhering to various language grammars but that's simply impractical. I prefer to fix bugs such as this from time to time rather then take forever to accept new languages.


We always use *singular* names with belongs_to!


lgtm, i'll cut a release later today.


@RPCMoritz please follow the Java/Scala convention of using camel case.


@RPCMoritz please do not create public methods other than the ones requested in this ticket.


@RPCMoritz please put camel case.


I don't think there's a need for the charity number here, is shown in footer. First paragraph on site should be concise as often used in search engine descriptions.


@RPCMoritz please adjust the spacings:

```def histogram[N](values: RDD[N])(implicit num: Numeric[N]): Unit```


Oh, that's nice! Does it mean `<script></script>` blocks will be correctly recognized as JS now?
Could you make it also for CSS inside `<style></style>`? This fix would be great, imo.


Need to specify the `repo_id:` here too to connect it with the repo. Annoying, I realize. ActiveRecord can figure out the user_id column since we're going off the association on the user object but it doesn't know what Repo we're working with.


so since this is not specific to the SAM3X subfamily, please change SAM3X_RSTC_H to SAM3_RSTC_H.


fixed and folded up into the metadata elif.


@RPCMoritz to avoid code duplication, please compute the optimal number of bins first and then reuse the existing histogram function on RDDs, passing the optimal number of bins as argument.


I think I fixed all of these


Thanks for your information, I will remove these codecs supports by default.
At least, enable them help me to test these codecs implementation of videotoolbox hwaccel :)


Just commenting, you can't have two vhosts with the deferred listen parameter applied.


No point. The valid types are actually start with an uppercase, but our catalogs output them in all lower, so I had just left it as the actual valid and lowered it manually.


+1
I updated FFmpeg patch in this way.


Cool, I didn't know that. I've updated all messages to do this.


From what I know, musl is the only libc we support which does not define `__GLIBC__` (bionic is the only other popular libc I know which doesn't defined `__GLIBC__`, but we don't build against that).

Also note that musl doesn't set __MUSL__ or anything like that, so we usually test for __GLIBC__ being undefined.


@tombatossals  @jessertaylor please note


No reason, just derp. I've updated all of those calls to be one line.


This makes the changelog have the next version at the header, note this is only valid if `bump-@` is run .. otherwise make a manual fix to changelog.


Okay, I can fix that in the next tag.


Sympy versions don't match


@Tilka This seams to work quite well with my limited testing, a few days ago i forked the main branch of Dolphin and pulled this commit into it, and it made my dolphin bar usable on my Linux HTPC. Had to add an udev rule to mode 0666 on 057e:0306 to let my user be able to use the wiimote.


I didn't investigate it but it was looping here, changing type helped.


Ouais, pour ajouter ça marchait à merveille, mais pour supprimer, ça lançait des exceptions. Finalement j'utilise une ArrayList concurrente optimisée pour les lectures (ce qui est le principal dans le jeu) et un peu plus lourde pour les écritures appelée CopyOnWriteArrayList


So there's a kernel side to this too?


// On retail else if (attack_attribute >= 350), can be considered a typo.


Fix: Commit implements delaying autorefresh, not autorefresh.


Minor version Y (x.Y.z | x > 0) MUST be incremented if new, backwards compatible functionality is introduced to the public API.
So it should be '1.1.0'


Thanks. Added locally.


@Oksisane Why did you destroy this method? It's a nice thing to have.


Right. I'll update the VDA-VT creation in this way.


@smo-key It's the exact same as getRotation()


@Oksisane - How?  getRotation() returns a value in degrees.  getRate() returns a value in degrees/sec.


Here are the results of my [benchmark](https://github.com/jstedfast/MimeParserBenchmark):

```
Parsing startrek.msg (1000 iterations):
MimeKit:        0.6989221 seconds
OpenPop:        25.3056064 seconds
AE.Net.Mail:    17.5971438 seconds
MailSystem.NET: 26.3891218 seconds
MIMER:          76.4538978 seconds

Parsing xamarin3.msg (1000 iterations):
MimeKit:        3.4215505 seconds
OpenPop:        159.3308053 seconds
AE.Net.Mail:    132.3044291 seconds
MailSystem.NET: 133.5832078 seconds
MIMER:          784.433441 seconds
```

As you can see, MimeKit is quite a *bit* faster than anything else out there ;-)

AE.Net.Mail also doesn't correctly parse the startrek message, so there's also that...

IMHO, if an email library cannot parse a message and then write it back out again exactly as it was originally, then the MIME parser in that library is broken.

MimeKit can do that and in fact was designed to do so in order to properly handle S/MIME and OpenPGP messages (which those other libraries don't do so well).

MailKit is built on top of MimeKit to add IMAP support. Once again, MailKit gets everything right by using a tokenizer approach to parse responses from the IMAP server rather than reading the entire response into one big massive string and using Regex/Split/Substring/etc and so that is also a lot faster.

It also means that MimeKit and MailKit can properly handle messages with multiple charsets... can't say that about the other libraries.


Syntax error! Should be `has_many` not Has many. We can talk about ways to try out your code as you go to prevent these sorts of bugs creeping in.


they also need immunity against AoE-Effects and despawn at end....


@kyrylo: thanks.


`belongs_to` associations should always be singular not plural. (i.e. `:user`, `:repo`)


@Oksisane
You have to reset the offset, too.  Otherwise, getRotation() will be messed up.

The purpose of getRotation() is for firing events when you rotated x degrees (or -x degrees).
The purpose of getHeading() is for calculating the heading used for FOD.


`boolean normalize` is redundant.  If you don't want to normalize, just use getRotation().


>> it would warn about IrresponsibleModule Foo from foo/bar.rb.

Ok, now I know what you mean.:)

>> I remember @dkubb fixed that years back, with reek moving to parser gem it was reintroduced.

`reek` works on files (omitting STDIN and rspec matcher) and those files are parsed independently from each other, so `reek` has - unfortunately - no knowledge about foo.rb in foo/bar.rb. AFAIK this has been always like that, so I don't think this has ever *really* worked. From a user point you don't care about all that of course since for you it is just and rightfully broken.:)
Anyways, thanks for your explanation, I'll keep that in mind.


This won't work because as defined in the migration above Repos do not have a name column!


@Oksisane Ah, that simplifies things.  :sweat_smile:


This plugin is ready ? I can't wait.. x,D ( I am French ).


After pulling this commit(618271e7d5) when I try running `main.py` I get the following error:
![screen shot 2015-06-15 at 00 45 52](https://cloud.githubusercontent.com/assets/11275466/8150595/22147440-12f8-11e5-849a-c5aa0dcbacff.png)
I've tried running `pip install -r requirements.txt ` with no luck.
Checking-out the commit before that(825bcc1ca4eecd405) works well.

Has anyone had the same issue?


*PR #4697 might help provide some context here.*

Because we, in the standard viewer at least, want to be able to show the `indeterminate` loadingBar when `percent === undefined` (i.e. when the server isn't providing the `contentLength`). Whether or not that is as relevant for the B2G viewer I don't know, but it makes sense to me to keep this code the same.

*Ninja edit:*
 - Compare with the standard viewer: https://github.com/mozilla/pdf.js/blob/master/web/viewer.js#L717.
 - Also, try searching for `indeterminate` in the `ProgressBar` code, beginning here: https://github.com/mozilla/pdf.js/blob/master/web/ui_utils.js#L354.


Exactly. But you can choose whether you're using IPv4 or IPv6.


why is this still commented


@livingincode DWMR.


No not TOO, just either way is fine. Just saying it can be done from the kernel as well


Is there a need to ever specify `self._default_scope = None` ? I thought `attach()` and `IndicatorExpression` intuited a null default scope when wrapping objects without a `_default_scope` attribute.


@Raesu DWMR. Press on.


:trophy: merging tomorrow morning!


Thanks @gastaldi  for the remark.


FYI, a very large telco in Kista were interested in using karamel with openstack this week :)


Thanks!


Understand, and you are more correct than me when it comes to this PR. Your changes are best done now. Just didn't want you going deep into other areas that have been completely redesigned already.

As far as the new codebase, no, it's isn't kept up on github, which is why I suggested just dropping a line and asking about making changes that don't have a corresponding issue.

Thanks for your work. Very awesome of you.


Not sure what happened with the commit message though...


I was using the expanded form to support evaluation in SLIME so that I could use ``macroexpand-1`` for debugging. I have not been able to get ``macroexpand-1`` to work in the JSCL REPL. However, I will change it back to the reader macro syntax.


@marcinpaszylk, Thanks for the contribution. How would this be tested? Even better would be to provide a unit test or test app.


Upon further inspection, this is a duplicate of SolutionGeneratorTests.testInitialBoardGenerationWithinLimit and testInitialBoardGenerationCheckingSudokuRule. This can be removed.


Scott, does that answer your question?


Welche Releases eigentlich? Gehoert wohl eher in den Absatz drunter...? Was ist mit Freifunkern, die einfach OpenWrt nehmen und meshwizard installieren? Davon wird es in Zukunft (hoffentlich) mehr geben -- eine komplette fuer die Community-spezifische single-purpose Firmware zu bauen kann doch kein Selbstzweck sein, sondern eher ein aus Not (= 4MB flash) heraus entstandene Situation...


Ugh I forgot the archivesBaseName in build.gradle too? Great lol.


you guys, did a really good job .. :)  by adding Camera source code ...
I was able to successfully start camera & also to fix video issue ...
All my work will be pushed soon on my github !


``parse-docstring`` arugment unncessary. Remove in next commit.


Is there a better method to use then `#each`? Take a look at some of the available [Array methods](http://devdocs.io/ruby/array)


Nope no bugs this time :P
The only small problem is that the moths are probably going under water. I'll fix that tomorrow


The camera full work in auto focus mode :smile:


This line breaks callbacks for CLUserEvent - it's queue is null, thus NPE is thrown. Possible soultion - create new field CLContext field in CLEvent, create new constructor with CLContext instead of CLQueue; use it if queue is null, but context is available like in case of CLUserEvent; don't forget to init context in constructor with queue in parameters; and finally fix this line by using  context but not queue.


@Raesu The problem with the buttons not working lies here. The test should be `([title isEqualToString:kWebBrowserBackString])` and so on. The reference to self is not needed as title is the name of a parameter of the method and self.title is accessing a property of the class.


This patch doesn't even compile...


@doubleshow If I'm reading this correctly the passwords seem to be stored in plaintext, or can be retrieved in plaintext, which is a vulnerability. http://plaintextoffenders.com/faq/devs


OK, I see the branches in distros.karaf and the redundancy I created
(althoug it was there before). I just needed a place to store this as
the rundir is much more recent than the runner-folder configuration and
support is going to be deleted soon. Just replacing the runner-folder
config with the rundir needs to be tested. Also the documentation
currently just refers to the rundir and must be adapted..
BTW: the distro repos are new and the only ones using the branching
mechanism. For everything else, we have used subfolders/repos so far
(e.g. mw core/osgi/android).
I need some time for testing and modifying the docs.

Am 14.06.2015 um 19:49 schrieb Alejandro Medrano:
>
> I do not agree with this restructuring.
> The distro repositories are so that anyone could check them out and
> get going. The new structure you propose sets this objective a bit
> more difficult (now users have to go to sub folders and ignore, or
> copy other parts of the repository), plus it adds a new meaning to de
> repository (now it holds also generic folders, the generic minimal
> configuration folder).
>
> observe Runner-Folder/runner/configurations already contained a most
> recent and complete configuration folder, thus rendering the new
> folder redundant.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/universAAL/distro.pax/commit/66750b4944c748b10e8331c807bcc677d4a5d664#commitcomment-11670722>.
>


#%^#, typo, I see it...


adds HTML parser


Getting the hang of things


This resolves #2 - we can now unattendedly accept license agreement and start using Rest Api.


That will be removed, fix needed elsewhere.


I applied a fix.


Please mark constants "private static final" and fix the typo newLilyPondProjectFileUseUserTemplat -> newLilyPondProjectFileUseUserTemplate


Garbage returned is int(0), when RPC fails for example (no credentials, anything really).

But the general idea is that the specific coverage plugin is not under our control and it can return anything whatsoever.

And because this is a callback, we cannot catch the exception there. Maybe we can add the error message to be sent back to the main plugin, but even then a child plugin can call it with 0 - it might be worth protecting against.


Done :) https://github.com/helmutkian/jscl/commit/b6e3fbab955d55efc2a9c2f49ad0e01b433c4c14

Changed to function ``make-js-object``

````common-lisp
(make-js-object "x" 10 "y" 20)
````

I provided a separate helper function ``symbol-to-js-identifier`` for translating Common Lisp style identifiers to JavaScript, so

````common-lisp
(symbol-to-js-identifier :my-property-name)
=> "myPropertyName"
````

This function could be used inside ``make-js-object`` so that

````common-lisp
(defvar *o* (make-js-object :my-property-name 11))

(oget *o* "myPropertyName")
=> 11
````

However, should we provide a way of overriding this? What if the user wants to map symbols to JS property keys in a different fashion? Secondly, this functionality should also be provided in the externalized version of ``oget`` so that

````common-lisp
(eql (oget *o* :my-property-name) (oget *o* "myPropertyName"))
=> t
````


Yes, that was the intent.

On Fri, Jun 12, 2015 at 1:59 PM, Ronald Holshausen <notifications@github.com
> wrote:

> @abhayachauhan <https://github.com/abhayachauhan> Good pickup. I think
> the idea was to have a match element to tell what type of matcher to use,
> so the correct example should be: {"match": "regex", "regex": "\\d+"}
>
> I'll wait for @bethesque <https://github.com/bethesque> to comment before
> making the change
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bethesque/pact-specification/commit/e1354c758dcc8a549db0cb05e61db015f20ca179#commitcomment-11647232>
> .
>

--
Bethany Skurrie
Software Developer

DiUS Computing Pty. Ltd.

*where ideas are engineered *Phone: +61 3 9008 5400
Mobile: +61 415 413 713

www.dius.com.au

This email is intended solely for the use of the addressee and may contain
information that is confidential or privileged. If you receive this email
in error please notify the sender and delete the email immediately.


Weird, in my local branch was `#endif` rather than `#endef`, must have happened when creating the pull-branch and manually editting the patch or something like that... Thanks anyway!


I have changed those files for the very reason I can read SDC sources in mono-d and now you remove them >-<


From what I can tell, ``declare`` isn't supported correctly yet, but this is future-proofing for once it becomes available.


but they did not compile sdc correctly.
At least the one for libd-llvm needs to be auto-generated to link with the correct llvm-libs


@Dicebot you might want to use sdc32-experimental
... then I had a reason to maintain it again :)


@Raesu Remove this test and see if panning then is smooth and visible as it happens.


Ah, right. I must say it was hard to realize without two renderers to test this out.

About the comment , I still thought certain settings were common to every renderers, but again, you are right, it doesn't seem so.

I can move the all the video settings back to only renderer dependent stuff for now. Would this suffice?


:+1: LGTM, thanks for the patch marv


This is precisely the discussion I was hoping we could have.

Right now classes set _default_scope to one of three values: Staff, Score or None. What's interesting is that classes never set _default_scope to Voice. Because it is probably the case that Annotation._default_scope should be None while it is probably also the case that, for example, Articulation._default_scope should be Voice.

What I did in the recent series of commits was just to set the _default_voice of all indicators to None unless Staff or Score was specified explicitly instead. Doing that gave us some regularity in the indicatortools API. And we're now in a position to decide what None vs Voice (and related decisions) should mean in the model.

So what do you think?

(Oh, and I think the answer to question you actually asked is that ``attach()`` and ``IndicatorExpression`` were in fact treating the lack of a _default_scope attribute as equivalent to _default_scope being set explicitly to None.)

So what makes sense next? I guess one way of motivating the question would be to ask which classes should be required to implement a _default_scope attribute. Which, bizarrely, is something we've never answered in the model.

As a first approximation, it seems to me like the answer is that any custom classes that we expect to be passed to attach() should implement a default scope. Yes / no?


This needs to be interesting so I can create a pull request. huh?


The commit message is a huge understatement.
You changed much more then just the location


Congrats on the 100th Commit :)


Plots 1:4


Why should you not stack metal detectors?


Please follow the Eclipse UI conventions and write "Custom template:"


So I can apparently compile it, but not run it.


How can we found the payload variable ?


@thanhdut DWMR. Much better - worth the effort.


что "&" это тут делает, Зачем ?


См выше


Tranquilo.
Que bom que deu certo.


I'm not sure about the side-effects of this in NuttX. We need a different call to abstract pthread create calls, because tasks and threads are not the same in NuttX. While the memory is flat, file descriptors are like on UNIX, hence we need to differentiate between tasks and threads.


you edited edit only?


Should add to the docs that if calls were stalled and you set the bottleneck to unstall, then it flushes all previous calls inside this function synchronously (so it doesn't do this in a thread).


Don't import things.


Bad whitespace


Shell commands don't work in windows :(

You want to use shelljs? http://documentup.com/arturadib/shelljs#

Script section in package.json  will look like this: ```"build": "shjs ./scripts/build.js",```

Example: Changing the **build.sh** file to **build.js**

```
require('shelljs/global');

rm('-rf','lib');
exec('babel src --out-dir lib');

mv('lib/react-entry.js','./react.js');
mv('lib/react-native-entry.js','./react-native.js');
```

Purdy. Save the planet, Bring cross-platform to the masses!!


Denis, here was one of them. I can't seem to find the other (too many hits when searching my Inbox).

The code below resulted in 0 rather than throwing the exception.

    try {
        SafeInt<unsigned __int64> a(ULLONG_MAX);
        a -= (__int64)-1;
        cout << (unsigned __int64)a << endl;
    }
    catch(...) {
        cout << "Overflow" << endl;
    }


Cant change the type of this field, people are dumb and use it directly. Try finding a way to wrap the field in the Set interface. Haven't looked at TLogHashSet yet to see if it's compatible.


Again, whitespace and imports. Makes it really hard to maintain across MC versions.


Again field type change, ESPECIALLY the protected ones, we know those are used poorly by people.


This plugin is read ? I can't wait... :O


accepted this pull request


Bah! I totally forgot about it (that was your first PR!)
Shell.js looks good. Do you want to do a PR porting scripts to it?

One important caveat is I want lint or test failures to stop `prepublish` script.


These comments arnt strictly needed, as obvious things that forge edits are fairly obvious because of the nature of .patch files.


This plugin is ready ? I can't wait... :P


Awesome!


change double quotes to single quotes on L114 to be consistent with the rest of the code

does this need to be JS, or can it be done with PHP?

(sorry for the late reply, been on the road/camping for the last week and without service)


Haha!  I've been heads down on project for work, and I was like Nooooooo.... :)

Sure, would like love to.  Let me work on it.


@loganlinn or anyone else -- do you know how to do this safely in clojurescript (go from record name to map constructor)?  I think this will fail e.g., if only the record name is referred, and possibly in other cases.


ormap


these are in racket/file


filename-extension in racket/path


drop this and use #:extra-files-paths in serve/servlet


There is kits ? When a player is tp against another, he has a stuff ?


I suggest reading the docs for web-server/dispatch, you have half of it and it would be a lot easier than this


I see. In POSIX, the px4_getpid() could return the pthread ID instead of the px4_task ID. Then it would be safe to call from a pthread or px4_task.


The plugin 1v1 is ready ? I can't wait... :P


could you add a "// fall through" comment at the end here please.


Let's call it **contentType**


@Hiradur Ok tnx
@Max98 I cant comile it after commenting out that line

```
home/slobodan/rigs-of-rods/source/main/main_sim/MainThread.cpp: In member function ‘bool RoR::MainThread::SetupGameplayLoop(bool, Ogre::String)’:
/home/slobodan/rigs-of-rods/source/main/main_sim/MainThread.cpp:667:18: error: ‘initMatManager’ was not declared in this scope
   initMatManager();
                  ^
/home/slobodan/rigs-of-rods/source/main/main_sim/MainThread.cpp: At global scope:
/home/slobodan/rigs-of-rods/source/main/main_sim/MainThread.cpp:1493:33: error: no ‘void RoR::MainThread::initMatManager()’ member function declared in class ‘RoR::MainThread’
 void MainThread::initMatManager()
                                 ^
										make[2]: *** [source/main/main_sim/CMakeFiles/RoR.dir/MainThread.cpp.o] Error 1
make[1]: *** [source/main/main_sim/CMakeFiles/RoR.dir/all] Error 2
make: *** [all] Error 2
```


... and **fileExtension**. Not using abbreviated names ftw.


LGTM with variable name changes. Tested with the basic file types we care about?


The year is 2015 and this file is copyrighted by you :)


ascus me sir but u da typing


You didn't understand me.
Comment this one:

>
RoR::Application::GetContentManager()>AddResourcePack(ContentManager::ResourcePack::BEAM_OBJECTS);
		//Before loading standard materials
		initMatManager(); <--- this one
[code]...


This is gcc specific I think, so you may have problems when porting this to other platforms


Instantly share images, videos, and other files on the web. The way it's written now implies that images and videos aren't files.


roger, my next pass will put everything into ForgeChunkManager


Can you put these after the android imports


What's the relationship between `close(t::Timer)` and `_uv_hook_close(t::Timer)`?  If I call `close(t)`, is `_uv_hook_close(t)` called as a callback?  Vice versa?

In https://github.com/JuliaLang/Compat.jl/pull/103, I just copied these, but trying to make sure that's okay.


> I agree that disabling this rule doesn't seem ideal, but why does it seem like the wrong thing to do?

"strict" is intended to replace "global-strict", so you the old setting shouldn't have to appear anywhere (even in a disabled state).


forgot to rename this use of the variable slot, didn't you @Aaike ?


This plugin is ready ? I can't wait... :P


Yep, sounds like it.

`shrinkwrap` is the `Gemfile.lock` of the NPM world, but even more important, I'd argue, because of the way Node does recursive dependencies, and also, how fast the ecosystem moves. It's really the only way to ensure your code is running in a controlled environment (ie. against the code you think it is).


This variable can be "final"


Not sure swallowing the swapcontext error is the right thing to do?


Looks fine to me, it is indeed an improvement.

Next person may merge.


I get an unused variable warning here because FCHECK gets compiled away


If we're changing this commit to be read only, then we don't need an options menu for editing.


make this const?


This can also be final (I like making variables final wherever possible)


Also for codebrainz/geany-plugins@f2c22bf49b7e27a65a135edb4b4aebbe592b9509


As above unused variable warn here


"or other files", then?


@puzzle1536 I see your code *ported* to s3ql:
https://github.com/krnflake/s3ql/commit/7fc6c597e994225f791bd179761514f0391399a3#diff-b6eeb11455e480e12c4bac395e246d8fR120


...and another warn here


This can be final, and it should be called "notesColumnIndex"


Issues Numbers: #1 #2 #3


Final


https://github.com/rime/home/wiki/RimeWithSchemata

该句与教程不同: fluency_editor v.s. fluid_editor


Hey....lets stick with "FIBRE" not "FIBER"!!! :-)


These can all be final


I don't think either "and" or "or" is more correct than the other. Go with "or" if you prefer.


did you mean to spell butter this way?


no. it ha been discontinued


I can change it, but it's a worse name.

I suggest you read up on https://talks.golang.org/2014/names.slide#1. It's more idiomatic Go to keep variable names short when they're declared and used in a small block of code. Longer variable names are better when the distance in code between declaration and usage is larger.

`ext` is `filepath.Ext(fileName)`, which is pretty clear. It's declared within a few lines of used. Making the name longer wouldn't help readability.


Calling `close` requests the timer to be shut down, and when it is (usually the next time we hit the event loop), _uv_hook_close is the callback that gets called. It's called by jl_uv.c.


the answer


These next 4 functions aren't needed any more


I think I would write offer/answer, but I see we already have a convention, so ok


Oh, I missed the suggestion in your original comment, I thought you just quoted what it said. "Instantly share images, videos, and other files on the web." sounds good to me.


I guess through testing, I see that `t.handle` is `NULL` after `close(t)`, which hints that this hook is probably called.


Let's remove the edit view for now and make it read only in this commit.


Go? What's a Go? I thought this was Java... ;)


@smo-key getRotation calls the gyrosensor.getRotation() which returns the degs/sec that the gyro sensor is measuring at that exact instant. getRotation is the velocity the robot is turning at, not the degrees rotated since the last call.


Hey karen, no mams! Borra esta linea!!! Jajajajajjajajajajajaja!!


This file can be removed.


Why use a set if you are going to iterate over it? Just use a list.


This seems to make a=imageattr, mandatory. Did we have consensus on that?


I can tell. :)


Ahhh come on!


Can you put this in alphabetical order (after episodes, before overview). Thanks.


@wilr: Can you remember what the reasoning for this was? It's resulting in the ampersands being escaped in the HTML of [api:] links, which causes the version= setting to be ignored and some links to not work (due to the class not existing in master).

For example, `http://api.silverstripe.org/search/lookup/?q=GreaterThanOrEqualFilter&version=3.1&module=framework` is being linked to from http://docs.silverstripe.org/en/3.1/developer_guides/model/searchfilters/


Not needed.


Yes, I think so. What you've done here was what I tried at first but I got a crash for the given reason. I don't think we should go further and not changing video settings is enough IMHO


This commit isn't needed any more for read-only.


:dancer: Good one