массив types - номиналы разменных монет
в методе change первый параметр - сумма, которую необходимо разменять,
второй  - количество разменных монет


using `bind()` is preferred over caching the value of `this`.


Không cần set value thủ công. Ở controller, set request->data, view sẽ tự động điền giá trị tương ứng.


thx that explains a lot - I have written a bunch of patches which generalize this - i'll send it over.
At some point, I will push all of those to my for-next on kernel.org and u can rebase the port on top of it ?
For now - just make sure the patches will work for you !


Hmm I've just had to build the 2.0 branch, and it seems this broke compilation on Windows.
VC++ gives me errors related to the use of bitset:

```
2>..\code\companion\src\eeprominterface.cpp(1369): error C2668: 'std::bitset<_Bits>::bitset' : ambiguous call to overloaded function
2>          with
2>          [
2>              _Bits=15
2>          ]
2>          C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\bitset(136): could be 'std::bitset<_Bits>::bitset(_ULonglong)'
2>          with
2>          [
2>              _Bits=15
2>          ]
2>          C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\bitset(127): or       'std::bitset<_Bits>::bitset(int)'
2>          with
2>          [
2>              _Bits=15
2>          ]
2>          while trying to match the argument list '(unsigned long)'
```


Rather than a global var, perhaps this could be a property on the instance?


Vet ikke hvorfor denne ikke har feila på develop - kan ha noe med caching av salt states, som blir sletta i clean bygget


ahah
I'm looking for the problems regarding to extending singleton classes.
I remember I read something about this in the past.


Even with something as simple as this, you might consider that there are more elegant (and useful) ways to generate a dict like this:
```python
VALUES = {"ace" : 1, "two" : 2, "three" : 3, "four" : 4, "five" : 5,
          "six" : 6, "seven" : 7, "eight" : 8, "nine" : 9, "ten" : 10,
          "jack" : 10, "queen" : 10, "king" : 10}

SUITS = ("hearts", "diamonds", "clubs", "spades")

def get_deck():
    deck = {}
    for suit in SUITS:
        for value in VALUES:
            deck["{value} of {suit}".format(**locals())] = VALUES[value]
    return deck
```


Is the goal of this line to copy the array?


Split SCC into separate free and 2 paid plugns.


My philosophy is to have least amount of platform specific code in ARC code and compartmentalize as much as possible. So this will cause some rework for you but I'm ready to help with it - but I prefer a clean / solid /non-hack code base which "I" can maintain easily over longer term !


Also, git doesn't seem to like indentation on transfer.


Right, and there are only certain ranges of values that you want to transform. It takes a little bit of binary math to nail down the correct range.


@KangOl had to be reverted, doesn't work during DB bootstrap when `group.user` doesn't exist yet. In addition, modifying the semantics of `_get_users_from_group` in such a weird way is dangerous and error-prone. Idea: add an extra wrapper method that filters the groups before calling `_get_users_from_group` and guard for the case where `group_user` doesn't exist yet.
Also please review the logic again and check whether this optimisation actually works when a user is removed from the employee group via the `m2m` on the group, and whether the `trans_implied_ids` thingy makes sense at all.


有兼容性问题，我merge完再改好了


Dùng FormHelper::input() để tạo radio group


Không cần đưa id vào form, dùng id ở URL


Thay Form->button bằng Form->submit


> Btw that would be really great to know what was the original motivation of @sindresorhus when he created this module and what he uses it for. Waiting for his response...

My main use-case was for accepting more than one level deep options in my modules. I want to mutate the supplied options object and hence need to deep assign it so I don't mock with the users original options object. But I do want it to work well in other scenarios too, so really appreciate yours and @schnittstabil work on improving it :)


Bỏ $this->Session->flash(), dùng Flash->render() ở layout


Bỏ phần pagination.
Nếu cần, CakePHP có cung cấp PaginationComponent và PaginationHelper để thực hiện.


It seems that author is on the github, but the code is not. I'll try to contact him like this:
@flok99, made a small patch for your "truncate" so it works  on OSX.


I hope you didn't put too much effort into this..  `moment(@exercise().dueDate).from(@serverTime())` did all that work with localization and less bugs ;)


Thay if(){..} bằng if(): ... endif;
Tương tự cho foreach.


Thanks!
where's the patch?

On Fri, Oct 9, 2015 at 9:27 AM, artw <notifications@github.com> wrote:

> It seems that author is on the github, but the code is not. I'll try to
> contact him like this:
> @flok99 <https://github.com/flok99>, made a small patch for your
> "truncate" so it works on OSX.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/artw/homebrew/commit/c4c77ef69d589a2decd6715a79af2fb767d223ed#commitcomment-13680188>
> .
>

--
www.vanheusden.com  www.slimwinnen.nl | www.smartwinning.info


HTML paragraphs need **always** be closed again. So please add the missing `</p>`


going back to ur detailed comment, is cpu0 special from h/w standpoint - does it have h/w threads ?
Also I presume the AUX_IDENTITY will return same value for all hardware threads of core ? Are Aux regs shared by h/w thread !


With homogeneous function tables there is no need for the 'signature check of the function at that index against an expected signature.' Sorry homogeneous function tables win on technical performance grounds.

Sorry, it is not true that 'Languages like C and C++ that compile to WebAssembly also imposed requirements, such as the uniqueness of function pointers and the ability to compare function pointers to data pointers.' These languages can meet their requirements with explicit code and don't require wasm core support. For example, C code could tag function pointers with the function signature in the hight bits, and then at an indirect call site could simply mask off these bits (a implementation using masking to avoid a bounds check gets this for free). If the producer wants to throw an error then it can also test the high bits. Comparison of these tagged function pointers would then meet the 'uniqueness of function pointers' requirement, and would still be deterministic.

I imagine that a lot of C and C++ code does not even need 'uniqueness of function pointers' between function pointers with different signatures and could avoid this overhead. What you are doing to imposing this overhead on all wasm code, which is not a good outcome.

The suggestion that implementations could implicitly maintain separate homogeneous tables derived from the heterogeneous table imposes a memory usage cost that scales as `num_functions * num_signatures` - this is simply unnecessary and a product of this poor design.

Sorry, I also don't buy deferring this until later. It's here now, add it for consideration now. It can all be re-examined when there are implementations to play with and implementation experience.

I would be fine with this if you wanted to try the heterogeneous tables in addition to the homogeneous, or if you wanted to add some better support for asserting that the tagged function pointer matches (a test_and_throw opcode) and an opcode to return a functions signature index to tag the function pointers. But I find the 'our way or the highway' approach grating.


Next step is working with `NSImage` and `UIImage`, which I got started with in 8020f84c8bc51092e2a22ba6439c44c9facf8246


erm, wrong order? first editor header then imports


ай-ай-ай


Ничего лишнего не должно быть в коммитах, они должны быть наичистейшими


Hello!

```patch
diff --git a/truncate.c b/truncate.c
index 3dcb0d0..53cb190 100644
--- a/truncate.c
+++ b/truncate.c
@@ -9,7 +9,11 @@
 #include <sys/stat.h>
 #include <fcntl.h>

-#include "error.h"
+#ifdef __APPLE__
+#define off64_t off_t
+#define stat64 stat
+#define ftruncate64 ftruncate
+#endif

 off64_t get_file_size(char *file_name)
 {
```


никакой лишней информации не должно быть


push it and I'll test too


typo


101162510-SubsurfaceOutwardAngleWarning-5016 (rglazer2) - Win64-Windows-7-VisualStudio-12: OK (1911 of 1912 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.95%25-yellow.svg)


Аня, ИзвИни (соблюдение правил тоже учитывается)


this looks like this:
```
[warn]          com.google.android.gms:play-services-plus:8.1.0: (Artifact(play-services-plus,aar,aar,None,List(master),None,Map()),/opt/android-sdk-update-manager/extras/google/m2repository/com/google/android/gms/play-services-plus/8.1.0/play-services-plus-8.1.0.aar)
[warn]          com.google.android.gms:play-services-plus:7.8.0:
```
and doesn't contain any info about where the dep is specified, I think we should keep the previous reporter helper.


Ah, yes. I see `TLOCK` this will fail. I just saw the use of `lockf` and I assumed it would just block. Just looked like it could be factored since the handling of the exception is the same. I suppose the outer try will still package up exceptions from the `openfile` into a Result monad.


если форма есть но напрмер она пустая, то обработка все равно пойдет. Какой смысл обрабатывать пустую форму?


Love it! :cat2:


isn't there any other way to do this? Like maybe, first look for the user, then check if they're not nil and later get the id?


I'm not sure I can follow.. There may be a misconception: Proposals that are currently in a CLOSED phase are still visible. This means, that even though a regional contest may be going through a judging phase, proposals would still be referencable. You'd only need to search for explicit deletions (/explicit visible == 0). This means, if a proposal from the creation phase doesn't make it to the finalists, it hasn't been deleted - although it's not in the finalist phase. Do you see what I mean?


括号少了个。


Here you are: http://www.doctrine-project.org/jira/browse/DDC-3944


please remove the `@` sign, as it throws an error


Done.


I pushed it to https://github.com/cschutijser/openbsd-wip for now. I just completed the testing, it works for me.


Not sure why below section is being shown in 'Green' and 'Red', the change is already there in SCHOLAR-3.x branch for Issue 7498


looks like i was reviewing this on wrong branch - adding here for record so I know i didn't miss anything :-)

This will likely go away with the more generalized patches I sent to Noam which enable reset vector to jump to @stext for master, and second_lines_of_secondary() for everyone else !


Hello
Is this newer than the one contained in the zip file offered for download?


vvv


@rhempel, re command chaining:
It can support any order. It work like this: each `set_*` method returns a reference to `*this`, so it can be immediately followed by any other method of the same class.


Everyone knows


Using outdated ConcurrectHashMapV8 library cause compilation errors with Java 8:
```
D:\OSMAND\Osmand\OsmAnd-java\src\com\googlecode\concurrentlinkedhashmap\Concurre
ntHashMapV8.java:3143: warning: Unsafe is internal proprietary API and may be re
moved in a future release
        private static final sun.misc.Unsafe U;
                                     ^
D:\OSMAND\Osmand\OsmAnd-java\src\com\googlecode\concurrentlinkedhashmap\Concurre
ntHashMapV8.java:3769: error: spliterator() in KeySetView cannot implement split
erator() in Set
        public ConcurrentHashMapSpliterator<K> spliterator() {
                                               ^
  return type ConcurrentHashMapSpliterator<K> is not compatible with Spliterator
<K>
  where K,E are type-variables:
    K extends Object declared in class KeySetView
    E extends Object declared in interface Set
```
of course you can patch current library version by renaming spliterator() function but may be move to Caffeine (Java 8 successor to ConcurrentLinkedHashMap) would be better solution? ConcurrentLinkedHashMap and Java 7 are not updated no more as you know.


I'll attach here my notes from 2012. Still interested to continue development some day!

Here is what works now:

 * It's possible to create several music themes in 'music/' folder via subfolders or alternatively using just different xml-files.
 * Theme can be read from xml file, but game should work also with plain music files.
 * It's possible to play different songs in different tech-levels (for example after windmills have been invented)
 * Game loops through all the songs. (No more infinite City Blues.ogg ;)
 * There is simple command for changing between the tracks. (Not implemented in GUI)

Future plans:

 * Transport control buttons ingame
 * There isn't any way to change music theme from GUI
 * Search music themes from LINCITY_HOME (or whatever it is)
 * Make xml file more flexible to allow grouping songs by tech level.
 * Playlist starts from beginning after visiting Options screen. This is because game restarts.
 * Randomize option for songs could be reasonable?
 * Maybe there should be a menu music different from game music?
 * A whole new musical score for Lincity-NG! I have few themes sketched already.


@lindamartin still no KEY :)


test


can you add a link here to giphy?


Сделал нумерацию строк


Hi simra,
Thanks for looking at this. The problem also applies to line charts. Incidentally, I'm new to this project, and I'm currently not sure what Gtk does. In any case, here is a short example:
```
let ticks = ChartTypes.TickMark(Interval=15.0)
Chart.Line [ for x in 1.0 .. 100.0 -> (x, x ** 2.0) ]
|> Chart.WithXAxis(MajorTickMark=ticks)
```
You should see that the tick labels and marks are out of sync.
Neil


I've sent an alternate patch which does this in a cheaper way - needs to be wrapped under
 #if defined(CONFIG_ARC_PLAT_EZNPS) && defined(CONFIG_PERF_EVENTS)


please use markdown numbered list


Hi,

found your change after your coment on issue #1784.
Does the addition of highway=services to POI (poi/poi_types.xml) and all related changes mean, that I should be able to select german "Raststätte"?

I ask this, because I cannot find anything like "<string name="poi_services">Raststätte</string>" in Osmand/OsmAnd/res/values-de/phrases.xml

greetings ,
Ratterkresch


This commit moves Mm Posts into its own folder within the components folder. Moving in the direction of having a folder for each component will allow for any extra files directly related to any single component to be better organized. We've already done this with the Countdown component, and now Mm Posts.

Within the posts component folder, we now have a folder for templates also. This is an ideal location for storing new named templates for Mm Posts. I've already started the party with a Simple List template. The basic structure of the simple-list.php file is a good starting point for creating new templates.


I think it was changed later


#szemfules :)


hier.


receive; ..which will come along with..


implement


`User.find_by_email` raises an exception when email does not exist


receiving


@rhempel, re `MotorHelper` idea:
The spec.json provides a [list of commands](https://github.com/ev3dev/ev3dev-lang/blob/98ce33f85366cc7b11065b2a9f7d7eeb109762ee/autogen/spec.json#L222-L268) for each of the motors. This list is used by both [C++](https://github.com/ev3dev/ev3dev-lang/blob/98ce33f85366cc7b11065b2a9f7d7eeb109762ee/cpp/ev3dev.h#L1034-L1055) and [python-boostc](https://github.com/ddemidov/ev3dev-lang-python/blob/6278794b2191ef3a0eb3a1551f8c984e425f76da/ev3dev/pyev3dev.cpp#L769-L788) to generate the helper functions. Same approach could be used in pure-python.


please use markdown list type :)


JSONKit will crash in the ios9, 'NSTaggedPointString', i can't resolve it


This will build the index even if you are doing ``--dry-run``, you need to add ``not options['dry-run']``


receive


Thank you for explanation. I cannot build the client on linux using your script because it takes too much space, is it possible to make it run from other than Home folder?


So ARC cache regs has a bit to globally disable dcache - why doesn't that work for you !


изменить /assets/image/bike...gif


and if u disabling the dcache, why not disable the config option. That way u will also void the cache flush routines from wasting cycles ?


@lindamartin nice improvement of interface and docs of NamespaceNameTree!


these can be squashed into the relevant bits - no value in calling them out seperately !
Also if you want to preserve bisectability (for CONFIG_ARC_PLAT_EZNPS) as ARC code has to work for all your patches I'd suggest put it under CONFIG_BROKEN in initial patch and then re-enable it in the end !


I don't see that as problem because actually you are not pushing anything to the TM when passing `--dry-run`, and having the TM might prevent possible unforeseen issues.


This is just merge commit (and GitHub probably shows other side of the merge than expected there). You most likely wanted to comment on https://github.com/osmandapp/Osmand/commit/51c14f1a9c3edfd3ecac358fe0ff827e832914f5


ACK !


@pynicolas yes indeed that's better.


@lubieniebieski ^

passed until discussed


+1, can guarantee it works in bigger apps as well ;)


latest


Thanks a lot!


O_O i'll be a left hand of you :D


``--dry-run`` == don't do anything.  That code does some interaction with the TM when you specifically ask it not to.  Why are we even having this discussion?


Per vedere il tuo sito online, occorre che il nome del tuo repository sia `nicola-antonelli.github.io` e non `nicola-antonelli`


Looks good. I personally use `reduce #'append` since I remember reading or hearing that is might be more performant since it doesn't relly on `appends` &rest arguments. Or was it the other way round? On an unrelated note, did you commit 710ea4d106683bc770597792cdeedfce80551614? How did it end up in the repo?


Hi @rbarretto18
Can you post the error here?


Do not make the entire class unsafe, use unsafe code blocks as needed. The users of this class do not need to know and do not care that the implementation uses unsafe code.


I still think the behavior of the function was wrong and needed to be fixed. However, knowing this, the fix would have been different. See https://github.com/odoo/odoo/pull/8974.


HI Levak !
Update game version for EU/NA/TR to 1.11500.3725.17500
https://www.warface.com/en/news/recent/warface-hotfix-8th-october-2015

And Update game version for VN to 1.1.1.328
http://warface.goplay.vn/news/details/2982/gioi-thieu-big-update-3-4


Still no word about how exactly organising imports - maybe you could export your current code style settings from Android Studio and share it somewhere in repo?


This should be inside the #else, as is now you unnecessarily pin the buffer on big endian architectures.


Unnecessary change.


AuthCodes.cpp  Cahnge         { 20490, 6, 2, 2, 'a' },  to   { 20574, 6, 2, 2, 'a' },


I would suggest `bundle install --path .bundle` here (which will be the default in Bundler 2.0), allowing us to `rm` in the next line...


`rm -rf Gemfile.lock .bundle` lets us delete installed gems without `gem uninstall` (and also keeps this bundle entirely separate from rubygems' system gems)


this is the hardest problem now, doing this right would probably involve pulling in something like https://github.com/softprops/semverfi


Do we really purge .gem files after 30 minutes? They will never, ever change, so I think we should probably keep them forever...


I didn't want to break the current naming convention for the methods. Adding an optional argument is less clear then a function which specificly says what its going to do. But Adding a `GaugeDelta()` method makes the regular `Gauge()` method less clear in its use. Renaming that one (IE to `GaugeSetValue()` is a breaking changes for current users.


Changed.


We should create a `bin/gemstash` that basically consists of `require 'bundler/setup'; exec 'exe/gemstash'`, and then document here that you can use that binstub to run gemstash as you work on it.


key が重複しているようです。


For the record, I think that `CONFIG_ATH10K_API2_FW` is overridden by `CONFIG_ATH10K_STA_FW`, which will pull this firmware: https://github.com/kvalo/ath10k-firmware/tree/da0f85d924226ee30c46e037120621c9e192b39e/main


The patch failed because of white space that were removed automatically by git. I didn't realized this and therefore made this empty commit. The build should be fixed with 6d89fff92a171327611f8b00de5a29948d845a89.


Point taken. I will fix this.


ddd


Heh. I compiled aosp) And I see bootloader logo only) No adb, nothing to debug. Fuck


Такие вещи типично означают что абстракция немного некорректна. Может быть, лучше вынести `io.emit(`broadcast:${event}`, data);` в отдельную процедуру и вызывать её напрямую.


:grin:


@nkovacs you mean they should be changed? if so, can you send a pull request?


       // Controlled remote craft.
            Vector3D position1 = MyAPIGateway.Session.Player.Controller.ControlledEntity.Entity.GetPosition();

            // actual character position.
            Vector3D position2 = ((IMyEntity)MyAPIGateway.Session.Player.GetCharacter()).GetPosition();

            // hardcoded position.
            Vector3D position3 = new Vector3D(123, 456, 789);

            var Distance = Vector3D.Distance(position2, position3);
            MyAPIGateway.Utilities.ShowMessage("SQR", "Range on {1} reads as: {0}", Distance, targetname);

            if (Distance <= 2500) { return true; } else { return false;  } //so did it come within our default range

2015-10-09 19:06:01:360 - Economy Client Log Started
2015-10-09 19:06:01:635 - HandleMessage
2015-10-09 19:06:01:679 - HandleMessage
2015-10-09 19:06:01:692 - Received - MessageConnectionResponse
2015-10-09 19:06:05:687 Error - System.IndexOutOfRangeException: Index was outside the bounds of the array.
   at Economy.scripts.EconomyScript.ProcessMessage(String messageText)
   at Economy.scripts.EconomyScript.GotMessage(String messageText, Boolean& sendToOthers)
2015-10-09 19:06:13:304 - Closed


This creates many instances of visitors while they don't seem to be really needed. Can this be avoided?


Nope, it's just a temporary fix to avoid the annoying message when setting the header and saving the .c files. In the current state, the plugin prints the date according to your $LANG and not in english like the norm says. I'm still looking for a complete working fix.


Fair enough, but more readable code is a thing.


Scripts need to be checked, yes, but let's postpone it for later. Please file an issue about it in the tracker


По кодстайлу, надо писать полностью как
```if(type == 'channels'){
	location.hash = e.target.innerText;
}```


Oook, mi sa però che questa roba è finita su gdrcd5.4 ma non su master. Quindi la nuova versione pubblicata oggi non ce l'ha


Not the files, the resolved dependencies (queries to the upstream).

The gem files are cached *forever*


Well yes, e.g. arxiu is catalan for file, according to google translate. I don't speak croatian very well, that's why I didn't send a pull request.


So now it is possible to read invalid decimal values instead of an exception being thrown?


BreakerYouko  merged 3 commits into GDRCD-5.4 from new_menu 12 minutes ago


1. I think this `:execute` should be `:call`
2. Why calling this function recursively? `findfile()` looks up 1st argument upwards recursively. (Please see `:help findfile()` if you're not familiar with `findfile()`.)


Ah, I don't know this option. Nice catch!


typo


一致条件を取る際に、比較対象が 1対多 場合、下記のようにも書けます。
list型 に対する in文 を使って...
> if prop.type in [dp.ofproto.OFPTFPT_INSTRUCTIONS, dp.ofproto.OFPTFPT_INSTRUCTIONS_MISS]:


First of all: is it intentional that you do not take into account the sizes of contained folders?

Secondly: We already have traversed the folder's children at this point (`sub_result = self._list_folder(...)`), so we have all the children's sizes. Hence we can reuse that information here instead of re-traversing everything, which would perform better.


Hi,

Not a problem. I set a default value to the parameter to have the effect of a reset_delay method but as you said, it's not necessary obvious for the user.


You don't need this for your usage


Nice - I like this kind of UX tweak :)


Ottimo, dopo questo danno che mi tocchi ricopiare le modifiche sul nuovo branch


You must specify the controller:
```
Router.route('/new/:postsLimit?', {
    name: 'newPosts',
    controller: NewPostsListController
});
```

```
Router.route('/best/:postLimit?', {
    name: 'bestPosts',
    controller: BestPostsListController
});
```


Quick! I am exporting the terms one last time today :)


Do we really need this link on the homepage?


@sindresorhus Not at all. I think it still fits your requirements nevertheless it's closer to the recursive version of the original `Object.assign`. Thanks @schnittstabil for helping me out.


Please don't do this - you can add include paths for platforms so you shd have
#include "plat/ctop.h"
In platform makefile,
KBUILD_CFLAGS  += -Iarch/arc/plat-eznps/include


This can't be correct.
You try to download the web page source without id, only from an regex? How should that work?


Done: http://git.dcmtk.org/web?p=dcmtk.git;a=commit;h=b3ffaff28fe390c5d5932ca00310f804027831c5
(I also included the credits to the commit message -- as always.)


What happens here if you try to initialise a pool of size 1 and it fails to create the 1 single job? It appears that init_size will be set to 0 and you end up with a pool with no jobs in but no error flagged. Are we happy with that behaviour? Will that cause an issue later when you always fail to get a job from the pool?


>  IMO subscription to the second source should happen exactly at the moment when the first one completes.

It was never like that though, this is not something new, the graph Concat was eager from day one. This was discussed before many times. Concat is handled as a fan-in operation and just like any fan-in (currently also any graph stage) always subscribes to all upstreams. The only way to make it lazy is to expose the act of subscription to the GraphStage API, which is next to impossible since subscription happens externally, in the materializer.

Also, we discussed ASAP vs ALAP semantics of onError before, and as far as I know we settled on ASAP (I can try to find the ticket, it was roughly a year ago) but I am not sure to what extent ASAP is logical.

So I am not sure what do do here.


`and request.values["recursive"] in valid_boolean_trues`


Congratulations on this patch !


ok grazie prof :+1:


Fixed BnetServer


Может переменную назвать `generalChannelIndex`? Все-таки там индекс хранится.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/22046/testReport/) for this commit**


you are breaking the ABI for rest of the world - offset of bta, lp_start changes now ! In user_regs_struct these need to go in the end !


Better to bring args into scope?
Also I'd prefer the comment like:
args = current.request.args
# Check if running from prepopulate
if args:

NB We do have another way to check prepopulate:
if current.auth.override is True:

However I think the ideal check is:
alert_id = current.request.args(0)
(The (0) doesn't crash if none exist unlike [0])
if alert_id:
    xxxx

The reason being that we need to also handle imports as well as prepops?
(& maybe even Sync) - hence the 'check prepop' check is insufficient


Здесь тоже непонятно что такое `select`, можно `hashChannelIndex`


Need the ifdef here !


Okay :)

By the way, I plan to release version 0.3 soon (in one or two weeks), which will contain your contributions.


Сразу после выделения стоит проверить, выделилось ли оно? calloc возвращает NULL, если не удалось


Would prefer we check job is not NULL before returning job->wait_fd.


То есть, если что-то пошло не так, мы говорим пользователю "Unfortunately, wrong input.", а дальше сортируем как ни в чем не бывало?


is this from source, or an estimated guess?


You should build both original and optimized versions!


По кодстайлу пишем явное приведение типов, так что `~` надо поправить.
И отступы тернарного оператора [по кодстайлу](https://github.com/shriming/codestyle#10-3)


Also, keep single `main.c` and build two different implementations.


Twitter to come then?

If defining a custom menu in the template (which I agree with) then I suggest removing the RSS channels from the core's cap module menu


Тут лучше проверить, существует ли массив?


Good ideas. I would recommend choosing a problem based on the availability of quality data that matches what you need to answer the question you pose. Most of the time of a data scientist is spent in acquiring and cleaning data which may not directly relate to the material in the class. Hence my recommendation to pick on the basis of the degree of availability of clean data.


More intuitive usage this way:

``` python
def analyse_recursively(files, path=None):
    if path is None:
        path = ""

    for file in files:
        #...
```

and then down below:

``` python
analyse_recursively(files)
```

Note: no `files = analyse_recursively(files)` because a return value doesn't make much sense considering you directly modify the provided `files` object within the method.


I am not sure if this should be like this


who added extra classes?


Also, try to think of how you would present the final results. WOuld it be a webpage or a graph or an interactive system ? Remember to scope adequtely as well.


Please leave the blank lines.


Again with the ASYNC_wake and ASYNC_clear_wake I'd prefer if it checked job is not NULL.


Niente di grave...sono sufficienti due comandi per mettere tutto a posto


Yap, removed! Thanx!


You should follow the consistent coding style. Please read this note carefully: https://github.com/embedded2015/phonebook/blob/master/README.md


Yeah, but was changed, later...


The full block here should only be executed if it's not a folder (it doesn't make sense for files), so just put an `else:` here and indent.


Use `astyle` to keep the consistent coding style.


Method BuildWorker::checkJobLimit does not have a Docblock comment.


Method BuildWorker::verifyJob does not have a Docblock comment.


Pandas work looks good. Great job !


Don't use meaningless names like `HL`.


1: M190 line is too long (63 > 50)


Non credo di conoscerli, ho ripristinato il branch dalla mia versione locale e sto facendo un nuovo pull


What is the difference between this and previous commit?


Please read this article to write down comprehensive GIT commit messages: https://robots.thoughtbot.com/5-useful-tips-for-a-better-commit-message


IMHO that patch looks nice! But I would not overwrite items() and instead use a different name which doesn't belong to the builtin dict as it might have side effects.


You should mention the effects after doing such modifications.


Waarom nog een een keer de google api client gebruiken? Je kan de locatie doorsturen via een handler.


Do you know I receive an email notification each time you make a new commit? :smile:


Alright, pushed to 9. Thanks!

On 09/10/15 10:20, Martin Trigaux wrote:
>
> Quick! I am exporting the terms one last time today :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/odoo-dev/odoo/commit/309e91050179bc2c542ec44220a29a65d71bc3a1#commitcomment-13680935>.
>


Don't say "version 2", which is meaningless from the aspects of software development.


Please keep the coding style. You should read `README.md` carefully.


You should mention your motivation and the reason why you would like to make such modifications in GIT commit messages.


prop_type に見つからなかった場合は、'Unknown' とかでなく、''(空) ということでしょうか？


Path needs fixing !


Bravo! So many insecurities removed! This should be merged with the main repo...


You should have fixed the grammar as well.


Please follow the consistent coding style! Read `README.md` carefully.


I did not. Anyway, it's merged now.


Windows.


Don't leave the dead code.


oh, it's a dependency of `python3-javapackages`. (`dnf repoquery -C python3-javapackages --requires --releasever=22`) -> we don't need to specify it in spec file then.


Awful stuff, man


It should be "initialized" rather than "initial".


Nice to know ! I'm looking forward ;)


But ok, I'll pull it.


The naming `Alph` is misleading.


Also this block needs to be under ISA_ARCOMPACT (or #else of ISA_ARCV2)
you need to CC Peter Z on this + some description of what the semantics are - we had a long discussion abt his on ARC HS on mailing list !


thx, Jeremy :+1:


Please use named params instead of v1..5


While compilation this appears:
1>..\src\combat.cpp(546): warning C4100: 'caster': unreferenced formal parameter

Is that right/fine?

This is the line:

void Combat::CombatDispelFunc(Creature* caster, Creature* target, const CombatParams& params, CombatDamage*)


Did you improve your GIT commit messages?


Sorry, wrong configuration before pull-request


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/22047/testReport/) for this commit**


Don't say "version 1", which is meaningless.


Do we want a range check here to stop someone specifying an idx that is bigger than the array and writing out of bounds? I presume you would need to check idx < CRYPTO_THREAD_LOCAL_TOTAL_VALS


5 seconds is a pretty long time, this means that on worse case scenario when exiting the application it could take at least 5 seconds before this is done ?
Maybe 500ms ?


You right, thanks.


This commit breaks CMSSW 5 compatibility. Please fix.


Should be PGM, PPM and PBM files


Please improve your GIT commit messages. Read this article carefully: https://robots.thoughtbot.com/5-useful-tips-for-a-better-commit-message


You have to mention the motivation and reason why you would like to modify the code.


No problem, dividing numbers was not the hardest part of this commit. Also, less buggy code is a good thing. :)


Don't leave the dead code.


@svanoort @recena I think this was the bug making `diff -u` not produce the correctly formatted patches for you


You should follow the consistent coding style. Please read `README.md` carefully.


3件、共通ですが、コミット文に Usage は不要でしょうか？


Presumably you also want to bounds check this array access (like comment below) to stop access of an arbitary memory address.


Why would you include?


Should be fixed in the last commit https://github.com/sbesson/bioformats/commit/11f5459c8af7a6907ca6bffa5f430201b12d20df


Don't you know the next line?


Python 3 emits warnings if you open a file handle without closing. The recommended idiom is
```python
with open(tmp_infile, 'w'):
    pass
```


entries -> entry... is there anythng more?


lock_path 가 static 변수라서, 동시에 filelock 함수를 호출하게 되면, racing 이 발생할수 있어 보임. static 을 사용하지 않도록 수정이 필요함.


`init()` should appear later.


Why?


Just saw that :) Looks fine then


Why is it ok to delete the sorting function?


co to za zakomentowany kod? :(


Please keep the consistent coding style! `astyle` is your friend.


As discussed with René and Raphael, this is not a good idea as this will lead to a wrong jet ID. The fractions must be filled properly instead of not using the assert.


@seamusabshere I thought this was to show that it does not extend an existing lock of the same key but different value (therefore "somebody else's" lock :smile:), it made more sense to me especially in the context of the two previous tests. You think we need another rename here?


The naming of `FREE()` is really misleading.


@rudolphpienaar Nice! Already got it working?


No. Either s/entries/entry/ or s/does not/don't/.


These calls to `list` are unnecessary; it's actually better to keep them as iterators. You just need to convert to lists when a list is required.


No, I didn't check. You're probably right, but that's way above my perl-fu knowledge.


Thanks, @colinbrislawn. I've already added this feature and other related features in code that has not been committed and published yet, so I will not include your PR. I will get the new code out very soon.


Using outdated ConcurrectHashMapV8 library cause compilation errors with Java 8:
```
D:\OSMAND\Osmand\OsmAnd-java\src\com\googlecode\concurrentlinkedhashmap\Concurre
ntHashMapV8.java:3143: warning: Unsafe is internal proprietary API and may be re
moved in a future release
        private static final sun.misc.Unsafe U;
                                     ^
D:\OSMAND\Osmand\OsmAnd-java\src\com\googlecode\concurrentlinkedhashmap\Concurre
ntHashMapV8.java:3769: error: spliterator() in KeySetView cannot implement split
erator() in Set
        public ConcurrentHashMapSpliterator<K> spliterator() {
                                               ^
  return type ConcurrentHashMapSpliterator<K> is not compatible with Spliterator
<K>
  where K,E are type-variables:
    K extends Object declared in class KeySetView
    E extends Object declared in interface Set
...
```
of course you can patch current library version by renaming spliterator() functions but may be move to Caffeine (Java 8 successor to ConcurrentLinkedHashMap) would be better solution? ConcurrentLinkedHashMap and Java 7 are not updated no more as you know.


Why change to `OrderedDict`?


the patch from Yuri seems to be forgotten to commit


The `_list_itemget` function was an addition that I made and then removed.


Review for interface/forms/eye_mag/view.php:
--Noted extract at line 65 (just noting). There may be a way to do this without extract since just appear to be building an associative array with the sql results; will look at this in more details the future. TODO.
--Notes extract at line 87 (just noting). There appears to case for this since guessing hundreds of variables. If this isn't overwriting any previous variables, then could use the flag to not overwrite previous values; all the more reason to deal with the extract in line 65.
--Will want to ensure the locked feature and esign feature play nicely together. TODO.
--line 127-128 translate
--at lines 131-136; for proper date formatting, use function in library/formatting.inc.php
--line 152 will require converting patient_file/encounter/new_form.php to the new security model in the future. TODO.
--line 156 need to translate and html escape.
--line 174, change to "echo xls()" and remove the 'e' parameter
--line 232 and 234, html escape
--line 243, attr($encounter_date)
--line 307, makes sense to only translate the "CC" and give it a translation comment, like {{chief complaint}}
--translate 308, 309 (see above comment)
--lines 318, 355, 377, makes sense to only translate the "Chief Complaint"
--line 326, 385 for translation constants, don't use spacing that is more than one space (this increases work of translation pipeline for "google doc" reasons).
--line 344, what is the OR? If it's a label, it will need to be translated.
--line 363 translation constant is not same as 326, 385. Shouldn't they be the same?
--translate line 412, 738, 739, 740, 760-765, 776, 781, 1112, 1202, 1203, 1245, 1292, 1293, 1295, 1340, 1341, 1512, 1604, 1615, 1699, 1703, 1707, 1725, 1731, 1737, 1743, 1749, 1755, 1759, 1837, 1873, 1875, 1881, 1883, 1889, 1891, 1897, 1899, 1905, 1907, 1959, 1965, 1971, 1977, 1983, 1989, 1993, 2062, 2127, 2131, 2135, 2160, 2166, 2172, 2178, 2184, 2195, 2271, 2317, 2326, 2535, 2540, 2949, 2976, 2981, 3001, 3038, 3094
--For the abbreviations, remember to give them a translation comment and ensure they are the same as used elsewhere in your code.
--line 1195 text($date)
--Do not translate manufacture names, brand names, supplier names, or drug names
--For 1781-1783 and 2014-2016 and 2217-2219, likely don't need to translate. What do the +1, etc do/mean?
--for 1790-1794 and 2020-2024 and 2224-2228, only translate the 'mm'
--lines 2331, 2341 use xla() for translation of the title in the html attribute
--line 2966 attr()
--Do not translate numbers (see lines 3020-3056)
--Line 3094 escape the parameters with attr()


asfghjk


А если это будет не Евгений Онеги, а например Война и мир на 35000 строк? Может лучше сделать его динамическим и расширять realloc-ом когда место будет кончатся?


omdat deze een deze locatie handler een veel hogere locatieupdate heeft.
en als de app afgesloten wordt gaat deze locatie handler uit maar de andere energie zuiniger locatieupdate gaat door.ik heb t gedaan vanwege battery issues.


Since this change we are getting java.lang.ClassNotFoundException: solr.DataImportHandler.


Out of curiosity, where are you calling this in your code?


Hi,
Please open an issue for the problem.


right, my bad. Accept is on it's way ;)


Thanks!Good luck with you!


This commit breaks CMSSW 5 compatibility. Please fix.


PLACEMAKER FOR ONGOING REVIEW


А что именно плохо? У пользователя умерла жена, собака и его дом сожгли?


**no-undef**: "expect" is not defined.


**no-undef**: "describe" is not defined.


**no-undef**: "before" is not defined.


**no-undef**: "it" is not defined.


**comma-spacing**: There should be no space before ','.


**semi-spacing**: Missing whitespace after semicolon.


**no-unused-expressions**: Expected an assignment or function call and instead saw an expression.


**no-undef**: "ReactDOM" is not defined.


That is not what `python-future` does. In Python3 if you remove some of those, it will fail. I haven't checked each of them yet.


só para saber, de onde vem esta música? talvez fosse bom sempre adicionar um comentário a explicar a razão de adicionar as música novas


@ruffle1986 Your are welcome. I already had plans, to fix the non-recursive `Array.isArray` case… Therefore many thanks for your reviews.


I've read this line three times now, those two conditions are the same, aren't they?


Fixes #241.


é o hino tema do acnac de tições de 2007
veio da separata desse mesmo ACNAC, ok vou fazer isso para as proximas
musicas que adicionar


I do not disable d$ altogether but with special bit we added I disable d$ for kernel I.e. for addresses in third giga like I wrote in change log.
I need d$ to work for FMT so I cannot disable it totally with generic control.

Sent by MailWise<http://www.mail-wise.com/installation/4> – Your emails, with style.:)

-------- Original Message --------
From: Vineet Gupta <notifications@github.com>
Sent: Friday, October 9, 2015 10:53 AM
To: EZchip/linux <linux@noreply.github.com>
Subject: Re: [linux] ARC: [plat-eznps] Disable D$ for kernel (7870ee3)
CC: Noam Camus <noamc@ezchip.com>

So ARC cache regs has a bit to globally disable dcache - why doesn't that work for you !

—
Reply to this email directly or view it on GitHub<https://github.com/EZchip/linux/commit/7870ee377b255e6f3c4067e1b35d7d522194381a#commitcomment-13680523>.


I don't really see the point of this commit. What benefits does removing `pathKey` have? And what benefit does renaming `poolPath` to `path` have?


Some test would have been useful. Otherwise the chance of a regression in the future will always loom over us.


I was having argument ordering errors in Python3, this fixed some cases. `dict` in Py2k and in Py3k don't work the same way in the constructor.


LGTM!


linha 24: "Deu" deveria ser "Deus"?


I tried your build.
File '/firmware/image/efs1.mbn'

The modem partition was not mount by fstab. You need to try mount it from init.geerhc.rc

wait /dev/block/platform/msm_sdcc.1/by-name/modem
mount vfat /dev/block/platform/msm_sdcc.1/by-name/modem /firmware ro shortname=lower,uid=1000,gid=1000,dmask=227,fmask=337


а где onegin.h?


Surely 'loops' means 'the number of loops', so -1 is a pretty funky value :) You could swap 161-170 and 180 with each other and change to 'loops_without_activity = 0' everywhere. I think this would make the counter more straightforward.

The reduction in pings LGTM.


Could you maybe not format code that you didn't change?


@brecky That array is build metadata sent to GM accounts ONLY, it does not affect being able to join realms at all.


**no-undef**: "expect" is not defined.


**no-undef**: "it" is not defined.


**no-undef**: "before" is not defined.


**no-undef**: "TestUtils" is not defined.


**no-trailing-spaces**: Trailing spaces not allowed.


done

On Fri, Oct 9, 2015 at 9:51 AM, Psaltério <notifications@github.com> wrote:

> linha 24: "Deu" deveria ser "Deus"?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/psalterio/repository/commit/34b387ed5ca28e282ae41fa0ab0b13fe3b51d3e7#commitcomment-13681547>
> .
>

--
//Tomas


consegues descobrir o autor da letra e da música?


То есть мы объявляем MAXLINE и тут и в main? А потом должны менять его в двух местах? Лучше вынести в хидер.


fixed missing sha check sum


@trbsi would you help with finishing translation? @mdomba have 10 minutes to help?


autor da letra e da música?


OK, I'll take a look at the example. You're dead on on the timestamp. Will do ASAP.


posso tentar perguntar a algumas pessoas no sábado, sim. esta é o hino tema
do ACNAC de tições de 2009.

On Fri, Oct 9, 2015 at 9:52 AM, Psaltério <notifications@github.com> wrote:

> consegues descobrir o autor da letra e da música?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/psalterio/repository/commit/45c9e957a156bb68233c7fd80f3ede9f6f26cab4#commitcomment-13681599>
> .
>

--
//Tomas


obrigado :+1:


Why doesn't removing this declaration break things?


N определяем и тут и в main.c? А если захотим поменять? Нужно в обоих местах?


Fixed in de54acf, thanks.


**no-undef**: "describe" is not defined.


**no-undef**: "it" is not defined.


**comma-spacing**: There should be no space before ','.


Example usage snippet:
```python
...
self.ST_LINK_CLI = 'st-LINK_CLI.exe'
self.host_os = self.mbed_os_support()
if self.host_os != 'Windows7':
    self.ST_LINK_CLI = 'st-flash'
...
```


to hard to read


commit comment test


То есть мы меряем в том числе и ввода? Зачем?


Do you think we need to call out explicitly within this documentation that calling ASYNC_free_pool when jobs are outstanding will not result in the heap allocations made during those jobs getting cleaned up (obviously stack allocations are ok as each fibre has it's own stack)? Essentially it is the users responsibility to complete all the jobs or they could cause a memory leak.


Its meaningful to have a name like PropertyModel instead of complaintTypeModel


当前时间（可以动的）


Hauska, en tiennyt että requirea voi (väärin)käyttää noinkin.


Второй ввод? То есть мы сравниваем сортировку двух разных массивов? И в чем смысл?


Good point about making the original unclear! How about adding both and we can mark the current `Gauge()` obsolete to be removed after a bit? It's a simple find/replace change for users, so I'm not fussed with making the change. Plus from experience and talking with others, gauges aren't used nearly as often as counters/timers.

Since `SetValue` sounds more like you're setting a value, maybe call it `GaugeAbsoluteValue()`? Bit longer though.


ЕСЛИ В ТАРИФЕ НУЖЕН РОУТЕР ???  СДЕЛАТЬ OS_TARIFFS_EQUIPMENT


What is the reasoning about hard coding the executor, rather than letting the user supply it using implicit?


показывать кнопку ДОБАВИТЬ для добавления оборудования на узел


why testing if you already create a state set in the constructor ? https://github.com/cedricpinson/osgjs/commit/7251fa869cde43740c08fae5d8879ce2d45b5ee5#diff-06be1c5d3b73034200024f5682678400R31


CREATE TABLE IF NOT EXISTS `os_switches_connections` (
`id` int(11) NOT NULL,
  `account_id` int(11) NOT NULL,
  `centre_id` int(11) NOT NULL,
  `object_id` int(11) NOT NULL
) ENGINE=InnoDB AUTO_INCREMENT=3 DEFAULT CHARSET=utf8;

--
-- Дамп данных таблицы `os_switches_connections`
--

INSERT INTO `os_switches_connections` (`id`, `account_id`, `centre_id`, `object_id`) VALUES
(2, 8, 7, 18);

--
-- Индексы сохранённых таблиц
--

--
-- Индексы таблицы `os_switches_connections`
--
ALTER TABLE `os_switches_connections`
 ADD PRIMARY KEY (`id`), ADD KEY `account_id` (`account_id`), ADD KEY `centre_id` (`centre_id`), ADD KEY `object_id` (`object_id`);

--
-- AUTO_INCREMENT для сохранённых таблиц
--

--
-- AUTO_INCREMENT для таблицы `os_switches_connections`
--
ALTER TABLE `os_switches_connections`
MODIFY `id` int(11) NOT NULL AUTO_INCREMENT,AUTO_INCREMENT=3;
--
-- Ограничения внешнего ключа сохраненных таблиц
--

--
-- Ограничения внешнего ключа таблицы `os_switches_connections`
--
ALTER TABLE `os_switches_connections`
ADD CONSTRAINT `os_switches_connections_ibfk_2` FOREIGN KEY (`centre_id`) REFERENCES `os_centres` (`id`) ON DELETE CASCADE,
ADD CONSTRAINT `os_switches_connections_ibfk_3` FOREIGN KEY (`object_id`) REFERENCES `os_objects` (`id`) ON DELETE CASCADE;


**comma-dangle**: Unexpected trailing comma.


**no-unused-vars**: PropTypes is defined but never used


**indent**: Expected indentation of 6 characters.


Какой абонент сделал - "Оплата услуг абонентом" ?
Неудобно оборудования добавлять на узел?
Когда отрубают коммутатор - отрубить абонентов
Абонентов необходимо заного подключить если коммутатор был отключен


**valid-jsdoc**: Missing JSDoc @returns for function.


**Eslint**: 'return' outside of function


**no-shadow**: Component is already declared in the upper scope.


**no-warning-comments**: Unexpected todo comment.


**no-extra-semi**: Unnecessary semicolon.


**no-undef**: "ReactDOM" is not defined.


**no-unused-vars**: ReactDom is defined but never used


**yoda**: Expected literal to be on the left side of !==.


**no-unused-vars**: event is defined but never used


**yoda**: Expected literal to be on the left side of >.


**no-shadow**: LoadingBar is already declared in the upper scope.


**semi**: Missing semicolon.


**no-shadow**: ErrorCenter is already declared in the upper scope.


**no-shadow**: MessageCenter is already declared in the upper scope.


**no-unused-vars**: children is defined but never used


@lindamartin It seems that you didn't add any new tests to track these cases.


That's true, must have accidentally duplicated it and didn't spot it; I'll remove it again.


there seems to be a quotation error. Shouldn't the line be:
```
throw new Error 'Expected uiScroll in form of \'_item_ in _datasource_\' but got ' + $attr.uiScroll
```


А что произойдет, если вместо а подать 1+2? Все ломается. Переменные в макросах нужно экранировать скобками


**valid-jsdoc**: Missing JSDoc @returns for function.


**no-shadow**: Component is already declared in the upper scope.


**no-unused-vars**: PropTypes is defined but never used


**no-extra-semi**: Unnecessary semicolon.


**comma-dangle**: Unexpected trailing comma.


**no-unused-vars**: ReactDom is defined but never used


**no-undef**: "ReactDOM" is not defined.


**indent**: Expected indentation of 6 characters.


**Eslint**: 'return' outside of function


**semi**: Missing semicolon.


**yoda**: Expected literal to be on the left side of >.


**no-warning-comments**: Unexpected todo comment.


You should mention the effect of your changes.


Don't mix two different changes into single GIT commit.


Sure, what you need


nit: Although this functionally looks OK, from a style perspective it's not good practice to rely on operator precedence like this; i.e. relying on the left side of the && being executed before the right. The following is clearer (I haven't checked it in a compiler!)
```
if (smc_fid == TLK_RESUME_FID) {
    if (!get_std_smc_active_flag(tlk_ctx.state))
        SMC_RET1(handle, SMC_UNK);
} else {
    if (get_std_smc_active_flag(tlk_ctx.state))
        SMC_RET1(handle, SMC_UNK);
}
```


Дважды вызов функции плохо. Ей приходится дважды делать одно и то же. Лучше вызвать ее заранее и сохранить структуру в отдельную переменную.


Please follow the consistent coding style. Read the file `README.md` and install astyle + Git hooks.


The function `FNV_1a_16` should be internal and marked as "static".


Do you have to expose the implementation details here?


Well, some strings are not translated into Croatian :)


Why not also upgrade Elasticsearch to 1.7.2?


Bots to the rescue


Instead, you can introduce a new helper function such as `init_phone_data` and invoke it at `main`.


@pynicolas PHPAstScanner is temporary (we will remove it after migration). This logic is used only here, so I don't think it's temporary.


@bbcallen Got it and thank you so much! I will change it.


Please keep the original coding style.


# SmartEditText-sizon
SmartEditText
A funny EditText view

![image](https://raw.githubusercontent.com/SeaSoon/SmartEditText-sizon/master/SmartEditText/art/SmartEditText.gif)

#How To Use#
``` Java
<com.sizon.smartedittext.SmartEditText
        android:id="@+id/et"
        android:layout_marginTop="30dp"
        android:layout_width="fill_parent"
        android:layout_height="wrap_content"
        android:background="@null"
        android:gravity="top"
        android:textSize="15sp"
        android:inputType="textMultiLine"/>
```

#License#
```
Copyright (C) 2015 sasakicks
Copyright (C) 2013 The Android Open Source Project

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
```


If you are about to implement your own resource deallocation routines, you should remove the above comment.


No problem, I'll do it today


This addresses https://github.com/nest/nest-simulator/pull/118#discussion_r41579153 and https://github.com/nest/nest-simulator/pull/118#discussion_r41579558.


I think you need to create a new release if you want them live.


а где проверка, что ввод прошел хорошо?


@pynicolas fixed


I know, I haven't fixed everything yet. I was fixing the core before fixing the interfaces.


Is is safe?


Oops, sorry about this. Just because of I could not find any document about this parameters from phantom doc properly, so I just made it first. Currently I just know v1 = host, v2 = port, v3, v4, v5 are quite unknown yet.

I think the default value for v3 is 'manual', btw


I see!


I've removed this in the other PR as well. I don't know how it got in there. Either while rebasing or some auto completion.


I didn't know this, thanks!


hue hue hue


Do NOT create a new release.
The current deployment mechanism casually wrecks the entire nopanic link system and the evaluation system (which is why we likely already lost a few results). Until we fix that, do NOT create a release (even though I deactivated the webhook, but still).

I just used my old manual "deployment". Works for now.


Changed in:
https://github.com/netguru/review/commit/a6d6a5768327a2f9e2c30838b7f5e90447bd4b66#diff-762ac17b6bde6786803af72a4727dec1


@Montellese How would this impact the sorting when not naming all the seasons ? My personnal simple use case would be to use this to rename the Specials season to something else but with the abitility to have it sorted at the end. As I do not like it being at start :)


Learning Github


**no-undef**: "describe" is not defined.


**max-nested-callbacks**: Too many nested callbacks (4). Maximum allowed is 3.


**no-undef**: "it" is not defined.


**no-undef**: "Component" is not defined.


**comma-spacing**: There should be no space before ','.


**semi**: Missing semicolon.


**no-undef**: "expect" is not defined.


**no-unused-vars**: store is defined but never used


**semi-spacing**: Missing whitespace after semicolon.


**require-jsdoc/require-jsdoc**: Missing @jsdoc for connectToStores


**max-nested-callbacks**: Too many nested callbacks (5). Maximum allowed is 3.


**no-unused-expressions**: Expected an assignment or function call and instead saw an expression.


**no-undef**: "before" is not defined.


**no-undef**: "stores" is not defined.


**no-unused-vars**: Simulate is defined but never used


**no-unused-vars**: isObject is defined but never used


ok, thanks!


```
if ( !isNaN( parseFloat( n ) ) && isFinite( n ) ){
    n = parseFloat( n );
}
```
?


true, doesn't make sense anymore


ナビゲーション中にstoppointを受信した場合，is_stopフラグの起立にしたがって，停止位置がwaypoints_に挿入される前に停止モードに入ってしまっていた（目的の停止位置に行かずにウェイポイントを通過した瞬間とまってしまう）ので，もうひとつフラグを入れて解決した．


Ok then lets fix the chagelog a bit so FMT is mentioned separately in 0-2GB line !

Also as you told me, FMT pages are magically mapped with NPS regs etc.
So the normal user space mappings effectively run with dcache off. You understand that with CONFIG_ARC_HAS_DCACHE there will be needless cache flush ops etc which u can simply avoid (and I bet gain 5 - 10 % more at say hackbench)

Perhaps the what u want is !CONFIG_ARC_HAS_DCACHE but dcache still being enabled.


But anyways that is an optimization we can do later as well. So ACK for this - although i want the changelog to change a bit as suggested !


Thanx, it was my mistake with header :)


So newstore won't support ssd journal + hdd data model?


See https://github.com/JetBrains/kotlin/commit/8ae9903992d2e404c392c08cc1c27ac0f10c0fe8?diff=split


@vilchik-elena This logic may stay here but we may have to move it to handle metrics related to comments when we remove PHPAstScanner: https://github.com/SonarCommunity/sonar-php/blob/master/php-frontend/src/main/java/org/sonar/php/PHPAstScanner.java#L192


See one of the other in-flight PRs that adds a better publish helper. This gets a bit easier.


so between STACK_TOP and CMEM there is unused gutter of 1MB.
It would been better if CMEM start was also aligned to 2M (PGD) so in future we could provide some mechanism to map those to user space just like normal pages !
I think this is not programmable - etched in silicon ?


Hope that life is still full of opportunities. No.


LSSB


post to zendesk


bra jobbat grabbar


I think the majority of platforms will support this. With OS-X are they just warnings? There are a couple options if we run into platforms that don't have it:
*) We can use the NULL implementation (so they don't get async support, but at least it compiles)
*) We can (possibly) implement some assembler to do the same job. We would have to look to Andy Polyakov for this, but it may be worthwhile anyway even for platforms that have these functions - I suspect we could get even faster context switching.


Snyggt jobbat!


Ahh, yes. They probably should be.


@pynicolas indeed. Then we'll see that later, during metrics migration.


you can remove this file (and all other visual indicator commits) in the revised tree


values 1 and 2 in a numpy array result in a discrete variable with 3 values


this commit can be removed in final revised tree !


Hi Paul,

I just wanted to congratulate you on your one-year-long streak of commits and contributions on GitHub.

Cheers /u

![One year of pfalcon](https://i.imgur.com/rgU2YJP.png)


This may not be a useful answer, but why are you still using JSONKit? `NSJSONSerialization` has been available since iOS 5. There are no iOS devices you should be supporting that can't use it. This library hasn't been update since 2012. It's time to let it go.


Setting `init_size` here is just to escape from the loop. It is the value of `curr_size` which is important - that's what is used in the `async_set_pool` call below. You are right though that `curr_size` could also end up being 0. This should not cause a problem because when you later try to get a job out of the pool it will see that there are no jobs available and that `curr_size < max_size` and so it will automatically create a new job at that point. Obviously if it still fails at that point then that is a problem - but there's not much we can do about that.


**Nothing**
*Just Adam commenting*


dcache enabled for FMT - so why is cache coherency not an issue - does h/w do that or applications are written to have private data only in FMT region which doesnt need cache coherency semantics !


@pynicolas Don't you think it's more readable to use PHPVisitorCheck if your "nodeToVisit" contains just one element. And you will not need to cast (as on line 57)


You don't need recursion here I think. And unless I'm mistaken you are not very error resilient here (what happens if there's path left but the matching entry is not a folder?).

Suggestion:

``` python
files = _getFileList(origin, recursive=True)
path = path.split("/")

if len(path) == 1:
    # shortcut for files in the root folder
    name = path[0]
    for f in files:
        if f["name"] == name:
            return f
    return None

node = files
while path:
    segment = path.pop(0)
    for f in node:
        if not f["name"] == segment:
            # wrong name => next!
            continue

        if not path:
            # no path left and name matches => found it!
            return f

        if not f["type"] == "folder":
            # path left but not a folder => that doesn't work
            return None

        # we'll use this folder's children as the next iteration
        node = f["children"]
        break
    else:
        # nothing matched the name, we can't find it
        return None

# nothing returned until now => not found
return None
```

(untested)


qt 5.4.2


@vilchik-elena indeed! Fixed it.


I won't create a new release. I just wanted to let you know in case you didn't. But it seems like you already do.

What gets wrecked? What link system do you mean?


posting


again


@pynicolas IMHO isFinalClass(ClassDeclarationTree) is better here


fixed issue


c'è un errore nella parola traffico


yep


We have had this debate recently within the team. When should we do NULL checks and when not. There are a *lot* of places in the current code where we do not, and adding lots of NULL checks  will litter the code without much benefit. We seem to be coming down on where we have a `FOO_xxxx(FOO *, ...)` function then we will assume that `FOO` is not NULL without an explicit check.


As above.


ok, I will be careful. Thanks a lot


Yes, ok. That might be wise.


this would be better if it were deterministic:
e.g. 60 ones and 40 twos

(now there is a 50% chance that the test passes even if weights stop working)


That won't work. I already updated it. Also pango is not released yet.


Agreed.


Good point. Yes, I will add that.


This method is not needed any more, right? `eZ/Publish/Core/MVC/Symfony/Matcher/BlockMatcherFactory::doMatch` does the exact same thing that is done on line 133 here.


@pynicolas If you will follow my advise on line 79, to be compliant, i would rename it to `isFinalMethod`


Does this mean that Semtech SX1301 Reference Board (usb-connected) can no longer be used with lora_gateway library?


And here you have the reason why `futureFilename` on line 211 was originally created explicitly on the local storage, not on `target` :)

The whole file upload to SD stuff is still a bit on shaky feet admittedly (with that detour over the `local` storage) and should be changed in the long run to be a proper `StorageInterface` with the streaming built right into it and all.


@pynicolas Please, move this message constant to class field


@Kelketek technically, this only tests that *some* user is selected, not the *first* one. Would be nice to fix it.


@alexsavio :+1:

I rebased and pushed to bcipolli/enh/py3k. If it's OK with you, I will force-push here, and this will become mergeable. Anything you'd like me to do before force-pushing?


I'm wondering if we should just be putting this data in the `<meta>` tag; will do some research.


Okay, just wanted to make sure nothing happened, I skimmed the deploying python script, so I'm at least vaguely aware of what happens. Although I didn't immediately see the problem: giving jekyll a destination dir makes it remove all its contents prior to building (which is not a bad idea per se). But in our case this static website is not the only thing residing in /2015, the (admittely shitty, but still existing) nopanic link resolution system (/2015/42 and so on) and more importantly the whole ese-eva (which obviously cannot be integrated into the static ese-website repo) have to live there, too.
For next year some solution for that will have to be found, but for now the best option might be copying from _site to the destination dir without cleaning it beforehand. @sebschrader or @JustusAdam should probably try to change the webhook script accordingly. Although I'm not sure if there will be many more changes this year anyway, so perhaps we shouldn't even bother trying to build this semi-horrible workaround.


@Kelketek I believe this would give exception if there are no peers/groups to review. This is unlikely condition of course, but what if a group contains of single person only? This will be the case and it will likely crash the rest of XBlock. So it would be best to add some protection for it.


`buildContentView` method should probably update the received view, instead of creating a copy. Besides, it creates a copy of `ContentView` while this class handlers blocks and locations too. Method should also probably be named `buildView`.


I'd rather check against count of children here instead of the size in bytes on disk. IMHO "empty" doesn't imply size, it implies count of children.


@Kelketek I believe you've added it on purpose, but it's not clearly visible from PR diff - could you please briefly outline the reason for these changes?


LGTM!


Of course, this is the most recent commit.


Empty line before that please, for readability :)


Oops. This needs to be `it.remove()`


Fixed, thanks


`var` (or it's global)


Why this change


Very nice! One bug I noticed:

When setting mode to "inherit" RTEs that are used as defaultExtras an text-fields are not loading anymore except the last on the page. When setting mode to "seperate" it works okay, so I can use this as a fallback for the time this feature is still in development.


Empty line before for readability please :)


This is still using the old rule name


`return $matcher->match($view);` ?


@nimaai please review

- made a 'common' base class (knows how to wrap the list)
- renamed 'MetaDataPresenter' -> 'MetaDataShow' (knows how to build a list of current metadata)
- made 'MetaDataEdit' presenter (knows how to build a list of current + *possible* metadata)


Subshell-free version:
wget -q -O /tmp/a2server-setup { ${A2SERVER_SCRIPT_URL:-http://appleii.ivanx.com/a2server}; [[ "$A2SERVER_SCRIPT_URL" == */ ]] || echo -n "/"; echo "setup/" }


Ive rebased


But anyways, doesn't matter since the method can be removed as per previous comment?


i want to add option attachment  send between agent  and client
please help and  advice/.....


It is in there, after I applied it I started to modify the tuning parameters as it was pretty complex and he spotted few mistakes (related to rolloff and plts) that I did not notice, clearly it was becoming a problem...


y used to have multiple values, now only 2


IIRC it still sorts by season number by default but maybe you can find another sort method that would cover your use case.


**no-unused-vars**: isDeployed is defined but never used


Класс - это конвейер, с помощью которого создаются объекты. На основе методов, которые описаны  в классе, каждый созданный с помощью класса объект имеет эти методы.
Объект - это все в руби. Объект является родителем класса Класс.
Экземпляр класса - это конечный продукт класса, который имеет свойства этого класса (методы).
Переменная экземпляра - это переменная, чье значение для каждого экземпляра разное, обозначается @ в начале.
attr_reader - создает переменные экземпляра и соответствующие методы, которые возвращают значение каждой переменной экземпляра.
attr_writer - тоже самое, только изменяет значение переменной экземпляра.
есть универсальный attr_accessor, который позволяет возвращать и изменять значение переменной экземпляра.


PR respinta, stringa già editata ed il suggerimento seguente è corretto nella sostanza ma non nella forma "da Mi Cloud" non "dal Mi Cloud", lo applicherò direttamente. Grazie


That's actually XW-440 `¯\_(ツ)_/¯`


@vilchik-elena An equivalent solution with PHPVisitorCheck would also need to check the kind, since ClassDeclarationTree is also used by INTERFACE_DECLARATION and TRAIT_DECLARATION.


@jkbradley
I agree. Comparing the iterators will be far more efficient however the dense matrix iterator and the sparse matrix iterator do not follow the same ordering. Take for example the following code and it's output.

val dm1 = Matrices.dense(2, 2, Array(0.0, 1.0, 2.0, 3.0))
val sm1 = dm1.asInstanceOf[DenseMatrix].toSparse
val sm1iter = sm1.toBreeze.activeIterator
val dm1iter = dm1.toBreeze.activeIterator.withFilter(p => p._2 != 0.0)
while(sm1iter.hasNext && dm1iter.hasNext) println(sm1iter.next() + " " + dm1iter.next())

Output :

((1,0),1.0) ((0,1),2.0)
((0,1),2.0) ((1,0),1.0)
((1,1),3.0) ((1,1),3.0)

For this reason directly comparing iterators does not work when the second operand is a DenseMatrix.
Instead I propose this solution in the last commit.

1. Check to make sure that the number of dimensions and number of non-zero elements match.
2. If it is a SparseMatrix use the activeIterator and compare both iterators with sameElements.
    sameElements will traverse both iterators and check that all elements appear in the correct order.
    This only works if the 2nd operand is a SparseMatrix.
3. If it is a DenseMatrix, then we use the activeIterator and use the apply function to check that the corresponding value in the DenseMatrix matches. Apply is O(1) time in the DenseMatrix implementation, while CSCMatrix is O(log numNonZeroInColumn). This is why it would be better to iterate over the CSCMatrix iterator.


I didn't mean to sound so dismissive. It did seem pointless to go back and forth about our ping delay settings. I don't actually care how much traffic you throw at Slack :)

Flooding channels: this is how I learned about disconnects. My bot wrote more than n lines in m seconds (n=m=50?) and I got regularly disconnected. This was a use case where it helped me (bot author) to learn about my disconnect quickly. LIMBO_DEBUG could set max_loops_without_activity=1 to improve that case.

Nerd correction: I embarrassingly miscounted the traffic. The ping call does a 101 byte write() and a 112 byte read(), but that's counting only TCP payload on top of which we have 32 bytes of TCP, 20 bytes of IP, and depending on the network maybe 14 more bytes of ethernet. The two ping packets are really more like 345 bytes.
345 bytes * 86400 pings/day * 0.07 $/GB for AWS traffic * 60k teams * 1% using limbo * 30 days/month = $19/month on slack's hosting bill. ¯\_(ツ)_/¯


Note that a2server-help.txt and a2server-aliases intentionally match line-for-line, so they can be easily edited side by side in two editor windows. Same goes for a2cloud-help and a2cloud-aliases.


Oh. I did not know that this things were also in this directory. Well, we have written a `rm *` with python into this deploy hook so we don't run into errors while copying the generated files to the target location. (permissions crap and so on)
Maybe we can get the script to overwrite the generated files and not giving a fuck about these permission errors. (Since this is only a subdirectory)
But still the no panic pdf link does not work in live. I get a 404.


@pynicolas Ok, agree, both have pros and cons


Original regex `/thank(s)?(ing)?(\syou)?(\sto)?(\s@)?/gi` matched simply "thank[s][ing]", which was giving a lot of false positives.

New regex `/thank(s)?(ing)?\s(you|to|@\w+)/gi` will only match "thank[s][ing]" if it's immediately followed by  "you", "to", or "@ and a word" - i.e. a username. Still gives a few FP, but not as many.


But shouldn't `pathKey` be set to `poolPath` in that case?


@guykogus I know, but my team started use in early, so.. need some time.
now, i still resolve it...


It's a duplicate of an earlier declaration, which I removed when I noticed it.


I removed the `rm *` a long time ago because jekyll does that on its own The effect however stays the same.


**no-undef**: "describe" is not defined.


**no-undef**: "it" is not defined.


**max-nested-callbacks**: Too many nested callbacks (4). Maximum allowed is 3.


**no-undef**: "expect" is not defined.


**no-undef**: "TestUtils" is not defined.


**no-unused-vars**: Simulate is defined but never used


**no-unused-vars**: props is defined but never used


**no-unused-vars**: renderComponent is defined but never used


**no-undef**: "sinon" is not defined.


**no-unused-expressions**: Expected an assignment or function call and instead saw an expression.


**no-undef**: "before" is not defined.


**max-nested-callbacks**: Too many nested callbacks (5). Maximum allowed is 3.


**semi-spacing**: Missing whitespace after semicolon.


Could you add a section about "Building the documentation locally" under the "Code" section of the docs? Thanks again.


Oh. Sorry. So then please ignore the last comment. ^^


add m before field no public


this change just made cordova mobile build failed to load avatars.

Meteor._relativeToSiteRootUrl in iOS returns ""


@darkalor Good point.


could you re indent the code at the same time ? I think in files it should be 4 spaces


LGTM!


@dshcheung hey, what an amazing commit, i love it

:love_hotel:


Thanks all for the input :)
@OpenERPJeff : Unfortunately there is no was to do that at the moment


I guess we could, it is more elegant way to handle this task.

Anyways, knowing cdh5 branch will become our future master maybe there is no need to apply it on CDH4 master right now. Unless there is some git magic that will allow us introducing this change in a reasonably easy way.


Maybe change the name? It can be confusing because it clashes with `ViewProvider` interface


It may also be the cause of #4240, with a possible fix in #4242.


Should be merged with master (see above, you changing translation keys back to strings).


Why did you change this from mortbay to eclipse?


This only really happened because I imported data but not users. Is it reasonable to assume that the user that submitted a form for a study still exists when that data gets exported at the end of the study? I think I'm just going to drop the assert.


@vilchik-elena The purpose of this check is to avoid the "final" modifier on a method which is (in a way) already final because the class in which it's declared is final. I therefore prefer to make it clear that this method is about checking whether the final modifier is present.


Good luck then :)


Definitely not, where is `extraClasses` defined? Never saw that.The default to add classes is the `class` attribute.


See my comment https://github.com/openaire/iis/commit/807c4cb943b2840abaeace2137f34131113a2c42#commitcomment-13682728


we've already discussed this. It's gonna be changed


Вот здесь как раз про группировку методов возникают вопросы. sort_by_duration у тебя выдаст отсортированные фильмы; а называющийся почти также sort_all_directors — никаких фильмов не выдаст, а выдаст режиссёров. Аналогично ниже: movies_by_genre — выдаёт список фильмов, а movie_by_director — нет, группировку фильмов по режиссёру (да, грамматически между ними конечно есть разница, но больно похожие названия).


this is not final, you should change mIcons


@vilchik-elena wouldn't isFinalClass be trivial?


My remark was attached to a particular line of the code, not the general commit. I was only referring to the ```CLI/prior.m``` function where you deleted all global variables including ```M_,options_``` etc and not only ```objective_function_penalty_base```


@pynicolas indeed!


I really should have thought this one through better :x


The scripts already will compile from source if they detect they're not on Debian7-x86 or Raspbian, in which case they get the binaries. While some packages for do exist in repos, for reasons I no longer remember I determined them to be either unavailable, incomplete or otherwise unsuitable in cases where I provide them myself. I do support making binaries available where possible because compiling The Unarchiver, in particular, takes a year and a half, especially on a Raspberry Pi.

One thing that will probably need to happen, at least for A2CLOUD if not also A2SERVER, is for the scripts to detect that it's Raspbian-Wheezy, now that Raspbian-Jessie exists. I'm imagining some things will break if it downloads the Wheezy binaries on Jessie.

While *BSD and other Linuxes might be legit candidates for A2SERVER, I have no burning desire to make it so, so that's your party. OS X would not be a good candidate, however, because unlike Linux and *BSD, It has no AppleTalk/DDP stack (except in obsolete versions, and Netatalk still doesn't support it in those),

.deb packages indeed would be nice, at least for the provided utilities. I never found a clear enough document on how to assemble them that made me motivated to provide them, and I liked the quick turnaround of making minor tweaks to the scripts and being done with it, best practices be damned. I also didn't want to have to worry about whether or not my repo was in their /etc/apt/sources.list, but the scripts touch so many other files that I guess one more is hardly a big deal. This is what David Schmenk does for the a2pi and gsport packages he provides.


Empty line :)


`var`


@spacecowboy Looks okay in principle. Please can you paste here the output from running those `lsof` commands with the relevant versions of `lsof`? I don't have a system to hand with a recent version.


**no-undef**: "sinon" is not defined.


**no-unused-vars**: renderComponent is defined but never used


**no-undef**: "describe" is not defined.


**no-undef**: "expect" is not defined.


**no-unused-expressions**: Expected an assignment or function call and instead saw an expression.


**no-undef**: "before" is not defined.


**no-undef**: "it" is not defined.


**semi-spacing**: Missing whitespace after semicolon.


**comma-spacing**: There should be no space before ','.


**no-unused-vars**: props is defined but never used


**no-undef**: "TestUtils" is not defined.


**semi**: Missing semicolon.


**max-nested-callbacks**: Too many nested callbacks (4). Maximum allowed is 3.


**no-unused-vars**: Simulate is defined but never used


**require-jsdoc/require-jsdoc**: Missing @jsdoc for connectToStores


**no-undef**: "Component" is not defined.


**no-unused-vars**: store is defined but never used


@pynicolas indeed, sorry for the noise :)


![screen shot 2015-10-09 at 11 48 28](https://cloud.githubusercontent.com/assets/306808/10390911/d521fb3c-6e7b-11e5-9d47-f8c7a63d88dc.png)


This comment isn't terribly helpful as it's fairly obvious that that's what we're doing from the code.


Curly braces please, unless it's only an assertment to quickly quit a function you should always use those braces, or you run risk of adding a line later, not noticing that the braces aren't there and then wonder why the line always gets executed.

Same thing a couple lines further down.


OK super


Ooops, sorry, thx for fixing it.


现在直播还不行点播可以了


Well I'd thought name / sort title would have solved this, but if name and sort titles are the same and only applied to modified ones, this whould leads to modified ones, sortables by alphabetical (not perfect but playing with starting space can help) in a group and the others unmodified in another group.

So I was wondering what was the point of the sorttitle in that case if we can't really make use of it.


No, it was my fault. I didn't pushed my changes in time. I had the same issue as you had with the package ;)


IMHO this is not functionally correct.


I'm all for better documentation/user manual/web site -- I think what's on the A2SERVER site is comprehensive and fairly well organized, but not attractive, and in dire need of visual aid. (The A2CLOUD web site/docs is much friendlier, or so I think.)

If useful, I can also provide the contents of the A2SERVER and Raspple II web site, which is just straight and simple HTML. (The A2CLOUD web site is WordPress, however.)


ooi v bad. remove this


Hi @krokodok - please report that problem to the TYPO3 core. It appears to be caused by attempting to use the `default` property on an RTE as you describe. When you enable `inherit` in Flux, the `default` attribute is used to define the value that is inherited so it gets displayed in the RTE - and that's the part that sounds like it's broken.


Yep, I couldn't execute the scripts since I did not have the rights to do it (I just changed the entire dirs in case there were others scripts...).

When the script (line `git new-branch flathead`?) executed I has a git error saying that I had to stash my changes:

```
Cannot rebase: You have unstaged changes.
Please commit or stash them.
Failed to update depot_tools.
```

So I couldn't execute the script properly.

Then I also had an `access denied` error when executing the `./build/createV8Dirs.bat`script.

In the end, when I tried to build the `deps\v8\build\all.sln` solution (Debug) I got the following errors:
```
Error	3	error C1083: Cannot open source file: '..\..\build\Debug\obj\global_intermediate\code-stub-libraries.cc': No such file or directory	C:\Users\Yvain\Documents\Flathead\deps\v8\tools\gyp\c1xx	v8_nosnapshot
Error	6	error C1083: Cannot open source file: '..\..\build\Debug\obj\global_intermediate\experimental-extras-libraries.cc': No such file or directory	C:\Users\Yvain\Documents\Flathead\deps\v8\tools\gyp\c1xx	v8_nosnapshot
Error	5	error C1083: Cannot open source file: '..\..\build\Debug\obj\global_intermediate\experimental-libraries.cc': No such file or directory	C:\Users\Yvain\Documents\Flathead\deps\v8\tools\gyp\c1xx	v8_nosnapshot
Error	4	error C1083: Cannot open source file: '..\..\build\Debug\obj\global_intermediate\extras-libraries.cc': No such file or directory	C:\Users\Yvain\Documents\Flathead\deps\v8\tools\gyp\c1xx	v8_nosnapshot
Error	2	error C1083: Cannot open source file: '..\..\build\Debug\obj\global_intermediate\libraries.cc': No such file or directory	C:\Users\Yvain\Documents\Flathead\deps\v8\tools\gyp\c1xx	v8_nosnapshot
Error	23	error C1083: Cannot open source file: '..\..\build\Debug\obj\global_intermediate\resources.cc': No such file or directory	C:\Users\Yvain\Documents\Flathead\deps\v8\test\cctest\c1xx	cctest
Error	25	error LNK1104: cannot open file 'C:\Users\Yvain\Documents\Flathead\deps\v8\build\Debug\lib\v8.lib'	C:\Users\Yvain\Documents\Flathead\deps\v8\src\LINK	d8
Error	26	error LNK1104: cannot open file 'C:\Users\Yvain\Documents\Flathead\deps\v8\build\Debug\lib\v8.lib'	C:\Users\Yvain\Documents\Flathead\deps\v8\samples\LINK	shell
Error	27	error LNK1104: cannot open file 'C:\Users\Yvain\Documents\Flathead\deps\v8\build\Debug\lib\v8.lib'	C:\Users\Yvain\Documents\Flathead\deps\v8\samples\LINK	process
Error	24	error LNK1104: cannot open file 'C:\Users\Yvain\Documents\Flathead\deps\v8\build\Debug\lib\v8_nosnapshot.lib'	C:\Users\Yvain\Documents\Flathead\deps\v8\tools\gyp\LINK	v8
Error	1	error MSB6006: "cmd.exe" exited with code 1.	C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V120\Microsoft.CppCommon.targets	170	5	js2c
Error	9	error MSB6006: "cmd.exe" exited with code 1.	C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V120\Microsoft.CppCommon.targets	170	5	resources
```


traepaca function already execute a rebuild operation, so i think it is a good idea to add an else sentence to avoid to repeat rebuild operation.


To be a little more verbose here the following error occurs while sending the question:

Request:
```
[
   {
      "result_first_version_path" : "@pn2",
      "result_path" : "@pn1",
      "method" : "POST",
      "body" : {
         "root_versions" : [],
         "data" : {},
         "content_type" : "adhocracy_core.resources.proposal.IProposal",
         "first_version_path" : "@pn2"
      }
   },
   {
      "body" : {
         "root_versions" : [],
         "data" : {
            "adhocracy_core.sheets.description.IDescription" : {
               "description" : ""
            },
            "adhocracy_core.sheets.title.ITitle" : {
               "title" : "Ist this question true?"
            },
            "adhocracy_core.sheets.versions.IVersionable" : {
               "follows" : [
                  "@pn2"
               ]
            }
         },
         "content_type" : "adhocracy_core.resources.proposal.IProposalVersion",
         "parent" : "@pn1"
      },
      "method" : "POST",
      "result_path" : "@pn3",
      "path" : "@pn1",
      "result_first_version_path" : "@pn4"
   }
]

```

Response:
```
{"status":"error","errors":[{"location":"body","name":"0.path","description":"Required"}]}
```


Shouldn't the parameter name change to `atom_group_ids` too?


@samdark done :)


edits


Yes, I'm the author of all scripts on the server.


Super ! bravo.


It was quite convenient for me to have them on svn but now, for the sake of code sterility, we could remove all `job.properties` and add them to `gitignore`.


See my comment https://github.com/openaire/iis/commit/8d0fb3681fb44ddb35aeade5534333a61ce94a68#commitcomment-13682938


I believe this artifact has been renamed. I cannot find 9.x in mortbay.


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: requirements


201:9: E126 continuation line over-indented for hanging indent
208:9: E126 continuation line over-indented for hanging indent


Итак, давай сделаем несколько финальных вещей!
1. Таки эта чортова группировка методов. Предлагаю следующую логику: у нас есть а) методы выдающие группы статистики (фильмы по режиссёрам, фильмы по актёрам, фильмы по месяцам) — они должны быть рядом и называться похоже; б) методы, фильтрующие по какому-то критерию (жанру, стране) — они должны быть рядом и называться похоже; в) метод, выдающий 5 самых длинных фильмов: пусть он будет называться непохоже на остальные, пусть по названию будет видно что он выдаёт не все фильмы, а только часть; да и сколько фильмов выдать пусть будет параметром метода, а не жёстко вписанным в код числом
2. ...и кстати пусть уж тогда метод группировки по актёрам (то что сейчас называется `movie_by_all_actors`) выдаёт всё же хеш `{актёр => фильмы где он снимался}`; это небольшое усложнение, но делает метод более полезным для других использований (мы всё ещё представляем, что мы пишем реальный софт «менеджер личной фильмотеки», скажем)
3. финальный вывод: пусть все фильмы выводятся одинаково, но разумно (в смысле, и «все комедии», и «5 длиннейших» и т.д. можно печатать как, например «1. Название (длина) — жанр»), а код хорошо бы чтобы выглядел как-то так (я вроде вчера это писал):
```ruby
print_movies list.longest(5)
print_movies list.by_genre('Comedies')

print_stats list.group_by_director
print_stats list.group_by_actor
print_stats list.group_by_month
```
Вот.


ありがとうございます！


Can you share the JSON string that you're trying to parse?


Yes, `piggybank` dependency covers that functionality.


Patch submitted upstream: https://github.com/heapsource/active_model_otp/pull/27


You may want to use `poolPath` instead of `path`, i.e. use http://localhost:6551/embed/meinberlin-stadtforum-proposal-create?poolPath=http:%2F%2Flocalhost:6541%2Forganisation%2Fstadtforum%2F


Okay nevermind. It was an error with the embed snippet for the create ui. It was given a path parameter which should have been pool-path instead. That explains also why @MagdaN commit solved the issue for us yeasterday.


![screen shot 2015-10-06 at 03 29 39](https://cloud.githubusercontent.com/assets/14892390/10391122/6ce26d74-6e42-11e5-81f1-3b94835d7930.png)


**no-undef**: "describe" is not defined.


**no-undef**: "expect" is not defined.


**no-unused-expressions**: Expected an assignment or function call and instead saw an expression.


**no-trailing-spaces**: Trailing spaces not allowed.


**no-undef**: "it" is not defined.


**no-warning-comments**: Unexpected todo comment.


**Eslint**: 'return' outside of function


**valid-jsdoc**: Missing JSDoc @returns for function.


**no-shadow**: Component is already declared in the upper scope.


**indent**: Expected indentation of 9 characters.


**indent**: Expected indentation of 4 characters.


**no-undef**: "ReactDOM" is not defined.


**indent**: Expected indentation of 5 characters.


**indent**: Expected indentation of 6 characters.


**indent**: Expected indentation of 8 characters.


**require-jsdoc/require-jsdoc**: Missing @jsdoc for componentWillMount


**no-unused-vars**: PropTypes is defined but never used


**no-undef**: "before" is not defined.


**no-undef**: "expect" is not defined.


Done


Add documentation for `smap` (type, how to use it...)


I added some new variable


We need to extend the Button component to work with an attached div - see issue #46


Do we need to cache these? Since the state is in the resolved counters, it doesn't seem like it matters much if a new instance is returned for each call to `counter(DynamicId)`. It seems like the main benefit is error checking, but I think that would already happen eventually on the resolved ids.


Completely IRL bro.

On Friday, October 9, 2015, Ian Page Hands <notifications@github.com> wrote:

> Is this real life?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/keithrob/openshift-node4/commit/d860c70966f9deef9671ba6a63b93883bdc8bb28#commitcomment-13687405>
> .
>


My 2 cents:

1. Acho que o mais recomendado seria ter um `README.md` em inglês e um `README.pt-br.md`, e um link no `README.md` para o `README.pt-br.md`, até por quê quem é da área de TI e não sabe nem ler uma descrição dessa… precisa rever seus conceitos;

2. Faltou modificar o `members.yaml`;

:P


@dimkr
This change was the only problem with mergeing your libre code into kernel kit. I'd appreciate if you could do a sanity check, but I think it's OK :+1:

Just successfully built 4.1.10 with an fdrv_nnn.sfs which I'm going to integrate in my next build (code is already in 3builddistro-Z to do that)


Good point! It must be an ambiguity, because methods can be implemented separately. Then you can ask why not to allow to distinguish between them by parameter names. I need to think more about it.


This is not the same code. Am not sure if it's supposed to be. The code here looks as though it is checking for items partially onscreen on left and right.

(This is not the same as the code in OnMessage which seems to be finding the first control fully visible, or checking if last focused was fully visible).


calc(var(--paper-toolbar-sm-height, 56px) * 2);


uhsuahsuahsuahusauh, eu segui o padrão dos projetos do THC, que estavam em pt-br, inclusive o `README`. Mas pode ser.


`calc(var(--paper-toolbar-sm-height, 56px) * 3);`


Is there a reason you want a method to return the parameters? I think your renames are a good idea, as they mimic the HTML names.

Could you submit a pull request?


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 62](https://teamcity.mesosphere.io/viewLog.html?buildId=56478&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 186 Build time: 00:03:55


Maybe catch `IndexError` if the variable is not in the storage map, to avoid a crash.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 63](https://teamcity.mesosphere.io/viewLog.html?buildId=56479&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 186 Build time: 00:03:57


oh ha! i really must have messed up my last rebase ;)


Something to consider (no, I don't work here ;)) is that ideally you could tell if the label needed invalidating based on m_info telling you that what you've set (after Parse) is different to what it already had, saving the if (get != new) set() stuff everywhere), as Get is potentially expensive. Simple way would be to store the label to parse in GUIInfoLabel and compare that in SetLabel.


@Xayrn

Hey man, here's a merged, buildable, rebased off master branch.

As it stands I think it's mostly just useful as a point of reference to finish getting everything working.  Specifically, I'm not sure if the way I'm getting the cards and items (equipments?) loaded in CardHunterUtils is kosher with your new design plan.

I'd like to start with this and peel out the UI bits that shouldn't be in the library and combine those with my App state class.

Plus there are just some bugs - nothing related to merge problems, so far as I can tell.  More like things looking in places for state where the state hasn't been initialized or isn't there anymore.


Why not include `data` here?
The result can be pretty much inconsistent if some nodes are in `profile.apply_time` and some are not, since the one without timing will try to print the data...


Add information about attacked planet, coord and how many time before impact


Let's also add a `// TODO:  ...` in the code here that references the GitHub issue.


Wonderful, Julian you're the best! +1


Add condition in case of specific case not take into account.


@alengel I'm not actually on JIRA, but sounds fine to me. I will probably go through the content again, but likely not until right before the next summer academy...


Pattern Matching when send mail


got KEY_RETURN compile error:
```
/home/bam/Arduino/libraries/HID-dev_2_4/src/ImprovedKeylayouts.h:48:23: error: 'RAW_KEYBOARD_RETURN' was not declared in this scope
 #define KEY_RETURN   (RAW_KEYBOARD_RETURN + 136)
                       ^
```


The entire ExeFS section got removed because it autodetects the segments now.
Because of the crash i have no idea why it crashed. Have not tested building .3ds because no one really uses them. The build was made using MinGW-w64 with -O3 and -flto.


Add the frequency defined in configuration


For all the problems of any global installations have vs local, isolated ones - version conflicts. `mocha-phantomjs` requires `+1.9.1` what if your global version is less then that? then due to a bug in 1.9.8 (see #167) I capped it at `1.9.7-15`, but a bug in `npm` wouldn't cap `peerDependencies` but always use `^` so then they failed (see #202)... Ugh. Search through the issues and see my `4.0` release notes for more information.

If you don't want to re-download `phantomjs`, use [mocha-phantomjs-core](https://github.com/nathanboktae/mocha-phantomjs-core) directly, like the `gulp` and `grunt` plugins do. That also works with phantomjs2.


JS界隈、結構 " より ' が好まれる傾向
FYI: http://cou929.nu/data/google_javascript_style_guide/


Maybe it's just me, but I find it more intuitive to group the getter and setter of the same property (here: handlers) together instead of first all of the getters and then all of the setters. I can also save one set of comments in that case because the other one already documents the property well.


;は大事 & Good!
これがないとconcatした場合などにうまく動かなくなることがある。でもLinterとか引っかかって辛い....笑


Set a new table to avoid "output field" to be hidden with authorized button.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 64](https://teamcity.mesosphere.io/viewLog.html?buildId=56480&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:47


Same here, let's add a `// TODO: ...` that references the GitHub issue.


`components endpoint` should probably be `component endpoint` (or if not, have an apostrophe before the "s").


Ditto, "server endpoint" or "server's endpoint".


Why does installation take a `<crate>` and uninstallation takes a `<spec>`? What is the difference?


@imcaffrey 刚刚发现这个问题。原英文文档就如此：

http://mithril.js.org/optimizing-performance.html

真是纠结啊


Despite all this documentation, I'm still not entirely sure who will set these handlers, server or component. Maybe this can be made clearer.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 65](https://teamcity.mesosphere.io/viewLog.html?buildId=56481&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 186 Build time: 00:03:56


@Abhinav1997 Goood)


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 68](https://teamcity.mesosphere.io/viewLog.html?buildId=56484&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 188 Build time: 00:04:00


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 70](https://teamcity.mesosphere.io/viewLog.html?buildId=56486&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 186 Build time: 00:04:03


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 67](https://teamcity.mesosphere.io/viewLog.html?buildId=56483&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 186 Build time: 00:04:04


我的解决方案，和你改的基本一致，就是sleep 5; continue，i设定为大于等于6，这样对脚本改动最少，又能达到目的。更新神速呀，赞一个！


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 73](https://teamcity.mesosphere.io/viewLog.html?buildId=56489&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 186 Build time: 00:04:15


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 72](https://teamcity.mesosphere.io/viewLog.html?buildId=56488&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 188 Build time: 00:04:15


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 66](https://teamcity.mesosphere.io/viewLog.html?buildId=56482&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 186 Build time: 00:04:17


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 69](https://teamcity.mesosphere.io/viewLog.html?buildId=56485&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 186 Build time: 00:04:20


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 71](https://teamcity.mesosphere.io/viewLog.html?buildId=56487&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 186 Build time: 00:04:25


Use
```
echo Lists::openList('ul');
  echo Lists::createitem('item1', ['class'=>'item-class']);
  echo Lists::createNavitem('link1', 'http://google.com', ['class'=>'link-class']);
echo Lists::closeList();
```

Renders
```
<ul>
    <li>item1</li>
    <li>
        <a href="http://google.com" class="link-class">link1</a>
    </li>
</ul>
```


Should that be `||` instead of `&&`?