TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1939](http://teamcity.monogame.net/viewLog.html?buildId=5010&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 366, ignored: 6 Build time: 00:15:40


There's an `underscore` dependency, you could make an array with `_.range(maxId+1)` then `msg.random _.range(maxId+1)`.


typo in 'handlers'


Zamiast isset proponuje uzywac empty


Ogolnie brakuje troche komentarzy, bo dzieje sie ciurek tekstu


Does that change where I should display negative numbers? as in, starting below the value rather than above it?


modb of the account. find them in core/kazoo_modb/priv/couchdb


Actually, now that I look at it, we may need the extra space when both negative and positive numbers exist.


No, this is not a big change. I just cleaned up some code with dynamic dependencies. See `load_task` which is essentially the only thing that changed. The reason the PR got so big is because of all the tests I added.


Posortuj setery, getery


hello


Test


Причем, больше чем нужно загрузится только, если есть недоступные треды.
Если 404 тредов нет, то очередь завершится сразу по проверке самого утонувшего.

Хм. Нужно сделать игнор тредов, если они уже были ранее помечены как `Not found` кнопкой `Обновить`.


`WREN_COMPUTED_GOTO` is always defined by wren_common.h, so I don't think it's good to add `defined(...)` here. I would rather see `WREN_COMPUTED_GOTO` defined in the MSVC project instead adding a special case to the `#if` here.


Tutaj powinienes uzywac getera


1. This is a public repo.  is there risk of having these keys be public?
2.  how do I use it :)
3.  what will it give me :)  (i.e,. what's the final url for the thing?)


worked for me...


Great
Eagerly waiting for LS :smile:


done


as HttpException plox


Wait, this appears to make the Tux64 repository as the URL for the submodule?

Am I missing something here?


Daj obsługe DELETE jeszcze


Только аккуратней. На старой макабе периодически вываливались ошибки 404, хотя тред был ещё жив. Не знаю как сейчас (ошибки 5XX точно запоминать не стоит).


Seems better to just merge the other commit, rather than doing work that will just need to be un-done later.  Is there a reason we can't just merge the other commit?  What's blocking it?


Shouldn't mraa_uart_get_dev_path() just return NULL in case of an error?


fast-forwarding develop to borshop-integration-46-feature/stderr_alert_output = 04bab411


the an is a typo.


tak samo as HttpException


Is there a way to do this through the MSVC project instead, maybe by passing a flag to the compiler? Doing that might disable the warning for all source files, but that seems okay. Same thing for the unsafe string function warnings you're ignoring.

I just don't like seeing source files cluttered up with compiler-specific things, personally.


You should really work on the format of the permissions. For example, it shouldn't be `healthplus.heal.command`, it should be `healthplus.command.heal`.


any idea when will launch a new version with this patch?


I wouldn't even mention the end of the file.


Hello @ronjouch. Shouldn't `varify` be declared in `devDependencies` ?


This commit needs to be reverted to solve touchscreen responsivene bug


I wouldn't mention the end of file here neither.


What is the scope of resetting the signal handlers?  I see that you do it when we want to execute a git remote command subprocess.  Previously, the signal handlers were responsible for tearing down the application and dumping the info.  It seems to me that it would be better to setup the handlers so that they register only for the main-thread, so we don't have to do this resetting.

That's probably outside of the scope of this commit, or it may even be a non-issue, but at first glance, it seems heavily laden with potential unexpected side-effects.


Whois has only one command. You should use `whois.command.whois` instead of `whois.command`. It doesn't make sense...


looks kind of weird to me:

```ebuild
DEPEND="
        ...
        crypt? ( app-crypt/qca:2[qt5] )
"
RDEPEND="${DEPEND}
        !net-irc/konversation:4
        crypt? ( app-crypt/qca:2[openssl] )
"
```

but if that's fine I can change it.


Dodac moze jeszcze warto sendmail?


so far I found 1 game which doesnt fit in this theory:
deathcox with dec_header 0x0001ff01
it writes to prot buffer 0x31c bytes, then reads 0x318, so there is no source data for stream restart, also wrong stream will be produced in this case after restart @ 0x200

but few more (alpilots, csmash) sets small size and it matches OK, so maybe restarts is disabled then dec_header & 0x00010000, or chunk size calculated differently in such case


?? wyjebane


Hm, I still don't see how this could matter. Neither value is special to Android, right?


I prefer to preserve the import ordering but it's not a big deal.


I am slightly worried that this might be calling the DB a lot, given a long array to sort. Since #get_position calls  KnownUri.find_by_uri(uri)[:position], there's a lot of querying going on.

You could make this slightly more efficient by grabbing _only_ the position, i.e.:

```ruby
def get_position(uri)
  KnownUri.where(uri: uri).select(:position).first.position
end
```

...But that is hardly an improvement in a large set. You could also cache all of the values beforehand:

```ruby
stat_positions = Hash[KnownUri.where(uri: data_point_uris.map { |d| d.statistical_method.to_s }).
  select([:uri, :position]).map {|k| [k.uri, k.position] }]
```

...And then in your sort routine, you could call on that:

```ruby
stat_positions[a.statistical_method.to_s] <=> stat_positions[b.statistical_method.to_s]
```

...That would only be making one trip to the DB (and a relatively fast one, at that.)

It's uglier, but most efficiencies are. :\

Thoughts?


I don't think it's worth deleting this.


I don't think you can depend on the `LOLLIPOP` constant since it won't exist in earlier SDKs.


This is interesting -- is this change relevant perhaps?


Looks like a pretty neat set of changes!


commenting on initial commit


ok


this should be: race['name']


No its not special android, i just used this randomly. I don't know which value also has the issue. Also please have a look here:
http://grepcode.com/file/repository.grepcode.com/java/ext/com.google.android/android/5.0.0_r1/android/support/v4/app/FragmentActivity.java/#150
This is some bit struggle stuff i don't really understand why doing this, nor i found something in the documentation about this.


Oh, please adjust it as you need, it only here for communicating it.


@jeanpaul1304 creo que debería ser así

Se tiene que colocar el atributo **lang** en la raíz del tag html con el idioma que se trabajará en el página para que las herramientas de traducción sepan que reglas usar y los sintetizadores de voz que pronunciaciones aplicar.


that seems fine i think, as long the test passes again, we'll just have to know that it's some what LaravelStyle specific functions, and test does not reflect the actually implementation.


Cool, thanks for doing this!


needs also the import:

```diff
-from subprocess import call
+from subprocess import check_call
```


@piengeng Travis passed just fine. we should be OK.


I have pulled your changes into a v0_09 branch but I'm putting it through a
team code review on our side before releasing.  It will probably go out
later this week.

On Sun, Jan 18, 2015 at 8:22 AM, Tom Hukins <notifications@github.com>
wrote:

> On Sun, Jan 18, 2015 at 08:15:27AM -0800, Matt Harrington wrote:
> > Yea, I may be mistaken. I'll take another look this next week. Do you
> mind
> > if it takes a few days to get to?
>
> Not at all. Thank you for taking a look at the patches so soon after
> I sent them.
>
> > I don't want to leave you waiting if it is blocking something on
> > your side.
>
> The module works well as it is, but I thought I'd send a long a couple
> of improvements I was able to make quickly.
>
> Tom
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tomhukins/perl-raven/commit/0efb8d7e5f72ba4103d831bc75dca90e7431a13e#commitcomment-9330773>
> .
>


sign ($ARG2 - $ARG1) * [4; 2; 1] <= 0
und in der nächsten Zeile analog: (...) >= 0
Hintergrund ist, dass die Dekorationen keinen ordentlichen Datentyp haben und als String kodiert sind.
Der o. g. Vergleich funktioniert nicht mit "ill", brauchst du das?


Have a look in the link above to grepcode, the whole handling there is done. So thats why i came to the conclusion that this is for most users useless. Also they might use the `android.support.v4.app.Fragment` as Google suggest to do always.


Requested by a few people, not even trolling.

On Sun, Jan 18, 2015 at 5:29 AM, Scizornician <notifications@github.com>
wrote:

> We're not having that as the font are we o.o?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/svivian/Pokemon-Showdown/commit/1589f5e02a09e3d707752eee6f3a0bfd52a0a346#commitcomment-9329973>
> .
>


Do you remember why we've needed that? Old code gone...


Thanks MrBaNkS :)


To be clear, I was talking about the resultant change in behavior to `curl_easy_recv`. This change also causes `schannel_recv` to return `0` with `*err == CURLE_AGAIN`. The return of `0` (and not `-1`, as before) in turn causes `curl_easy_recv` to return `CURLE_OK` instead of the expected `CURLE_AGAIN`.

My proposed fix:
```
  else if (*err == CURLE_OK)
    ret = 0;
```


Oh hell yes, maybe just assert for the platform int?


This did nothing to fix the problem.


I'd to apologize for the // comments here and there. haha, it's for later debug use. I'd no idea how to put all these into your existing core Align___ class. I'll give it some thought, maybe, when it's a bit stable later on.


That's way better than my proposal!
/like


i just had it because I had to override the method since I implemented the listener interface


¿ Ese 0 que sentido tiene ?


@piengeng don't worry - if you feel like grooming the code later, just do it and open a PR - and I shall merge gladly.


Probably worth adding java doc to setSubjectOnClient ().

Also would it be possible to create a new test SAM assertion so we can remove the LM and i4ce?


Hai installato archlinuc
:smile:


Usted dijo que lo hicieramos en cualquier lengua


@Razican this was just a proposal. I did this as a "least worst" solution because I'm personally bothered by the lack of empty line in the code blocks :confused:

@narfbg your opinion on this?


KK, good point...will add to later API


Yes I just write in the int, with a comment. Right now the whole project assumes at least `ICE_CREAM_SANDWICH_MR1`. But do you think this change is at all related to what fixed it for you? The other change, I'm not clear it would have an effect.


Should I just keep it as is then? Or maybe I could lighten the gray so the text has more contrast?


I believe this scales it to fit the _unit sphere_, not the unit cube?


We'd only ever want to reset the signal handlers in a subprocess, not the main process. You're right -- the signal handlers in the main process handle graceful shutdown which is vital so that we tear down the app correctly.

The subprocesses for git remote commands are different though -- they are just one-off throw away processes and it's fine if there is an unhandled exception in the subprocess since that will get passed back to the main process and handled there.

Another reason for disabling signal handlers in the subprocess is that there is logging done in the signal handler and we don't want two separate processes trying to log to the same file at the same time.

All of this is just a big workaround for the pexpect issue anyway. Hopefully pexpect fixes the file descriptor limit at some point and we can just revert all of this.


make it much better


This isn't a char* pointer


This goes back to the "K-V store API" discussion.


This is so we don't print out the debugging message if we're in the challenge stage of multiround authentication? We don't necessarily expect a password here...


This is not in conjunction with the fix, just adjusted for future. You can ignore it.


Yes I think NULL would be a better choice, makes error handling much easier.


DONE in 4bfb6effca548a57b8231686dafdcf11c72001e6


test


Show just do
this->HasStemDir();
(It would work, but write Parent::Method() when you explicitly want to call a parent method also defined in the child)


No... The use can set Auth-Type yubikey in unlang...


Might be good to add to this: “unless you’re using Test Kitchen, in which case it handles all of that for you”.


agree :-)


At a low-level, its gonna be a K-V store anyway, and I prefer to have that abstraction as close to the code as possible.


Блджад. Выяснил очень нехорошую весч.
Парсер `PagesLoader'a` заполняет массив `pByNum` загруженными постами.
В случае с проверкой страниц в избранном - это сотни "виртуальных постов", которых нигде нет, но они кагбе есть..
В ризультате после проверки кнопкой `Страница` крашится автопрдгрузка, не работает удаление из избранного, и прочие некошерные вещи!!!!!


No. I the real issue is:
```java```
int index = 0x0000c0de>>16;
```
The result is zero. Which later prevents invocation of `onActivityResult` on the fragments in the `FragmentManager`. I will check if this happens also on plaforms < 21.


Why change the variable names?


Chord* chordParent = this->IsChordTone() ?


We don't use this style (in modern code) anywhere else in the server?


Just why... You should never make spurious formatting changes, and mix them in with code to add additional functionality.


Isn't `parallels` a NumPy array? Can just use `astype` or whatever.


Blerg


Very nice, very nice.. Couple of things:

``[[ ]]`` is a bashism, so anwhere ``sh`` isn't symlinked to bash/zsh etc... that will fail. I am not sure you are using a comparison that actually needs it, so you could use ``[ ]``.

Also The ``sh -c ''`` isn't really needed, so you can simply drop it.

Then you may need to escape the ``$(git log --oneline $remote/$current_branch..$current_branch)`` with ``\" \"``.

The next thing is that you call ``git log twice``. It doesn't really matter, in most cases, since it will be quick in 99% of repos.

Now, ``git log``  also accepts custom output formats, for example ``git log --format="%h %s"`` which will print only the abbreviated commit hash and the subject, i.e. first line of commit message. That can save piping to ``sed``.

Anyway, I made a refactoring using an inline function definition and a closure. I also formatted it as nice as the gitconfig would allow:

```
	remote-difflog2 = "!current_branch=\"$(git rev-parse --abbrev-ref HEAD)\" ;\
                       f() { remote=\"$1\" ;\
                             echo \"$remote:\" ;\
                             log=\"$(git log --format=\" %h %s\" \
                                     $remote/$current_branch..$current_branch)\" ;\
                             if [ -z \"$log\" ] ; then\
                                 echo \" uptodate\" ;\
                             else \
                                 echo \"$log\" ;\
                             fi \
                       } ;\
                       for remote in \"$(git remote)\" ; do \
                           f $remote ;\
                       done;"
```


Нужно как-то удалять все загруженные объекты и очищать массивы после использования `PagesLoader'a`!


Not setting the Auth-Type was intentional... and if we were going to do it, we certainly wouldn't just tack it to the end of the control list.


Там нужно что-то с конструктором делформы придумать.


Как чувствовал, а.. Аще думал, не приведет ли полноценный парсинг к побочным эффектам.
У меня-то раньше просто применялась рекгулярка к загруженному HTML...


"Please check this module for possible problems."


This kind of resource-related methods are kind of duplicated all over some classes. This gets also repeated within amp-redis and amp-mysql. What do you think of creating a small Resouce class (within nbsock) to get this into a class with some easier to read methods?


> что-то с конструктором делформы придумать

Таки да.


Der Vergleich wird benötigt, wenn sowohl tightest als auch accurate Outputs geprüft werden sollen. Dann soll die Dekoration des Ergebnisses "zwischen" den Dekorationen der Outputs liegen. Die ill-Dekoration ist an der Stelle genauso erlaubt wie jede andere; ob sie in so einem Test Sinn macht weiß ich aber nicht.
Ich bin mir aber nicht sicher ob ich die Implementierung so in Ordnung ist, ich rede nochmal mit Marco darüber und frage ihn auch wegen der ill-Dekoration.


Yea but then one still has to remove the duplicates. `np.unique` exists; true enough. I actually find this here more readable (although the additional `list` conversion is not really needed and the `sorted()` call is just a safeguard ...)


+1 for namespace inexor::util::


Should have an extra space above the method definition


This one sheet, seems to be the demise for all texture pack artists.


+1
I also need uuids for the entity system


Ouch!


According to document this is a char * so we should free it, http://www.imagemagick.org/api/magick-property.php#MagickGetHomeURL


sp [vertices]


Okay, tanks


v.kind_of?(Vertex) is usually preferred, === for case statements.


Wait, but this will result in the same old bug again meaning a new TriggerDB object with a random name will be created, even for triggers without parameters?

Or did the context in which this function is used change and this is not the case?


alt_id could be an upro ID, so proCompile is incomplete.


"0" is the encoding you give the hardware for a dimension of 2048.


That sequence is flagged as compressed so, if the decompressed data is
0x318 bytes long, it doesn't make sense to think that the compressed data
is of the same length, so probably it's just overwritting the input past
the end of the real sequence. It's not necessary that other substream exist
after it.

I'm not saying that the proposed code be correct, just that that example
doesn't imply anything.

On Tue, Jan 20, 2015 at 9:17 PM, MetalliC <notifications@github.com> wrote:

> so far I found 1 game which doesnt fit in this theory:
> deathcox with dec_header 0x0001ff01
> it writes to prot buffer 0x31c bytes, then reads 0x318, so there is no
> source data for stream restart, also wrong stream will be produced in this
> case after restart @ 0x200
>
> but few more (alpilots, csmash) sets small size and it matches OK, so
> maybe restarts is disabled then dec_header & 0x00010000, or chunk size
> calculated differently in such case
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mamedev/mame/commit/cd61a2736d8149f6640a5756b83caae55b7b07ae#commitcomment-9363235>
> .
>


some comment


Все, что нам надо от парсера —, это `form.tNums`. Все остальное нинужно.


This change is simple enough that we could just redo it in the release branch without using a cherry pick.  It could then be merged from release into devel (again).  My main concern in doing it in release, though, is that I'm not sure how to test it or how risky it is/isn't.


Should not need explicit check. Sets can unconditionally add.

    result_set << v
    v.ancestors(result_set)

ought to work.


why do you need the view here ?


Тащем-то я хотел ещё поиск зарилить по содержимому постов.


That's because of the bug in line 160. We were creating a trigger object even when there is an existing trigger db.


"Manjši popravek" ni ravno najboljši commit message na svetu, je pa res, da si je za ta popravek težko izmisliti boljši message. Recimo "Avtorja nista več sladkorčka".


I would at least fix https://github.com/cbitstech/think_feel_do_engine/commit/0ef1757213685a6c9894ef394c81abccc3117bcd#diff-8ed5071c66ef369d302e7a65477770b1R242 before shipping.


This commit is indeed redundant. A better solution to detect the path to mtipli.dll using 'which vsim' was merged into master after I had created the commit.


commit should read "add include footer to default.blade.php


А, если на задел -то таки да. Пусть все будет. полезный конструктор. Может и зойпилим.
Но для проверки страницы в избранном надо все лишнее удалять.


Ah, OK. Thanks for clarification.

I just hope we have good test cases for all of those edge cases now :)


no, data is not compressed, its flag is 0x20000 (with 0x10000 selecting pack buffer size)


interesting way to cast, can also just do:

```
output = (<-chan interface{})(ch)
```


Now gives "CompilingError" fixerino pls...


I've been able to reproduce either the issue or a related issue on Lollipop fixed by this change. It's safe (or at least a different issue) < KitKat, because prior to KitKat it wasn't possible to reconfigure Bitmaps, so configurations were never mixed. As far as I can tell, all reports are from L or KitKat also.

Happy to change it though if you or anyone else finds an issue < KitKat.


Yeah, our tests caught this. There are too many edge cases.


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


Indentation


> `mock-javamail` deployed via special plugin

Yes, I guess you could do it this way. Using a true SMTP server would produce a higher-fidelity test, though, and without the need to deploy a plugin.


Cool! :+1:

I kind of like the docker image size differences for these:

	goapi               latest              8ea3b0fe6efb        9 seconds ago       679.2 MB
	curtlabs/goapi      latest              49445e150ef8        42 minutes ago      540.4 MB

goapi: using old docker image
curtlabs/goapi: current dockerfile setup


But I only want to make the recursive call if it is new.


Not sure what you mean?


Will have to add some turning in the headland ifr having 180 turns ...


can we remove unused namespace (xmlns:c="http://java.sun.com/jsp/jstl/core") from connectionmanager.xhtml


I believe that should read "... to `M-}` and `M-{` respectively"


Good point. Will edit.


This is more a way to bypass the fact that Track is immuable. This current_metadata_track is a merge of current_track and metadata.


yep, that's do-able.


I don't see that namespace in this pr.  Probably out of scope for this.


Took care of that in https://github.com/SimpleFinance/chef-zookeeper/commit/0537792a5ac6f70385b12acb674b4bf94626e179


Oops, right.


thank you <3


and probably isBlank() is better here since it checks that the string is not composed of white spaces.


Otherwise recursion grows as the cumulative sum of the depth of the dag.


good


Actually i sort-of do mind.¹ It is doubly un-Pythonic.
* The style guide says to, to use `if lang not in self.service_dict:`, and people call it more readable.
* When you want to return rather than throw the exception out, the Pythonic way would be
```python
try:
    self.service = self.services_dict[self.language[:2].lower()]
except KeyError:
    return
```

¹Not enough not to pull this. D’uh.


Variante mit ill-Unterstützung:
decorator_less_equals: decval ($ARG1) <= decval ($ARG2)

... und dann in test.yaml/imports noch diese Funktion einfügen:
%!function n = decval (dec)
%!    n = find (strcmp (dec, {"ill", "trv", "def", "dac", "com"}));
%!endfunction

Allerdings denke ich nicht, dass ill für diese Prüfungen relevant wird. Ob ill vorkommt oder nicht ist keine Frage der Genauigkeit.


Disabling the warning for every source file that includes "wren_value.h" was exactly my purpose, since the "zero-arrays" are declared in the header file itself.

However, yes. Generally speaking it's possible to disable the warning from the project file settings. But I would advise non to do it this way, for this particular case.


better do ```from django.conf import settings``` then it's possible to override this by changing it directly in settings.py (to access it later in the file it's just settings.WGER_SETTINGS[])


ok!


Yep, I noticed it.

However, the behaviour need to be force-overridden for the MSVC compiler.

Another approach would require to undefine the macro in the "wren_common.h" file just some lines below.

I'll give it a second thought and let you know. ;)


r+


Good to see --version getting more accurate. Ledger_VERSION_DATE needs bumping as well. (Ideally, automatically of course).


Ebgaza spuria sta matia


This PR is not yet complete enough for the build team to comment on.


@mledoze oops...


It's made redundant by the `import ctypes.util` on line 71 - but I could make it more explicit by changing L71 to `import ctypes, ctypes.util` if that's preferable.


Ok. Since its new .xhtml, I thought it would be good to remove if we are not using.


I reviewed and good with PR.


saw approval from christophermancini
at https://github.com/basho/riak-python-client/commit/f9acd6e17ba6cb08d6fee1ca9fe4630c20028460


merging basho/riak-python-client/feature/bch/tox = f9acd6e1 into borshop-integration-392-feature/bch/tox


basho/riak-python-client/feature/bch/tox = f9acd6e1 merged ok, testing candidate = da3f66e1


AArch64 immediate addition and subtraction actually support shifting the 12 bit immediate value to the left by 12 bits. So it would be possible.
On Cortex-A57 each MOV{Z,K} instruction costs a cycle in latency, and each basic ALU operation costs a cycle.
So if the MOVZ+MOVK+ADD(3 cycles) is less than the required number of additions then yes?
But a 6000 would use one MOVZ+ADD, so I don't think it would save anything there versus two adds?
negative numbers on the other hand would always require a MOVZ+MOVK due to sign extension, so two subtractions would save a cycle there in those instances that it doesn't fit in the negative in to the 12bit instruction encoding.


:confused: nobody does this


Hmm, maybe. I think the default is a pretty good default, and it works for the current issue that we're seeing (lots of open IPv4 connections). Do you mean we should choose something different than the default, or just specify 'inet' explicitly?


Isn't this an empty loop due to `aliases` being `{}`?
What is it supposed to do?


just ran this; should be vm.runInThisContext, not vm.Script.runInThisContext

I don't know why closure chooses the undefined latter.


nice to have you back working on pvr :)


Why? I do. And it's one less step for our customers to get up and running.


Ok, I used .inject because the array is composed of two element arrays and I need to average all the first elements.


Nobody puts this in the readme.


Also, @michaelwehrley  I check the array size in the method that calls this.


removed.


It's for easy copy pasting! :O


+1 for passing ttl!


@hshimo Does mounting the directory as NFS allow LevelDB to write to it? If so, send me your bitcoin address, that solved a problem I've been stuck on for a while.


... but it's not a requirement


Should have said `Streams.finallyFun`.


Yea. I should add a ARM64XEmitter::AddByAnyMeansNecessary function.
I'll do that later though


Yamux is in extremely wide scale production use in Packer, Terraform and Consul.


I can't actually, since I'm using offsetoff it gets really angry at me if I try.
Although I do the same thing in the ComputeRC functions and it doesn't get angry.
GCC is picky about the weirdest things.


Lmao, it's not the code that's bad, its spending 4 HOURS finding all the libraries for the gcc compiler. ;(


Yes, I know. I really wanted to ask that if You would develop it now, would You use it or is there currently a better multiplexing library in Go that You are aware of?


Why does one need to double click these forms?


By moving the constructor out of the loop, it re-uses the FreshRSS object for several users, and I do not think this use-case has been tested. It seems to be the cause of https://github.com/FreshRSS/FreshRSS/issues/759


As I wrote in the improvements section: one could do without the outer keyCredential, right? It only contains the nymIDSource in addition to masterSigned, and masterSigned also contains that nymIDSource. Strictly from a redundancy point of view. Signatures by the keycredential and the mastercredential, resp., would both cover the same stuff (the nymIDSource is signed by the master as well as the keycredential anyway).


Obljubim, da bom naslednjič bolje premislil.


I really don't know, but the first click does not register (at least in our system).


Because it is assumed that all the latest versions from framework-bom are to be used in tests, it is also assumed that calabash (dependency of common-utils 2.1.0, but not of 2.0.0!) is on the classpath. A better fix is to take into account the actual version numbers of the bundles on the classpath when computing the dependency tree in org.daisy.pipeline.pax.exam.Options.


I'm confused. A quick look at the current source says

        FLAG_COMPRESSED = 0x10000

On Tue, Jan 20, 2015 at 9:50 PM, MetalliC <notifications@github.com> wrote:

> no, data is not compressed, its flag is 0x20000 (with 0x10000 selecting
> pack buffer size)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mamedev/mame/commit/cd61a2736d8149f6640a5756b83caae55b7b07ae#commitcomment-9363697>
> .
>


None that I'm aware of, there may be. This fits our needs so I haven't checked for others.


part of IQSS/geoconnect#79


I don't have an exact date but it will be soon.


I think this looks like a nice feature to have, especially if it could be used somehow to provide type-specific code blocks in traits.

But I am really uneasy about assigning Boolean to Value.  Very confusing when you see Boolean and don't know if it's a SimpleType#Value instance or a type.  Yes, I know the compiler can keep it straight, but that doesn't mean I can.  (And I need to also.)


Sure thank You for this answer and this piece of software:)


[SideCI/RailsBestPractices] remove unused methods (User#find_first_by_auth_conditions)
reference:


No problem, it happens ;-)


I hear the words you are saying, and I'm going to let you finish. I'll write a helper function in the coming weeks and use it when I do it then. :+1:


some tests failed:
failure: http://buildbot.bos1/builders/python-client-test/builds/316


Sorry, I was not very clear. The problem isn't the `+=`, but the missing export. Here are `MAKEOPTS` would only include `-j1` for the time being inside `src_compile()`, but not inside e.g. `src_install()`. However, I was suggesting to move that line inside `pkg_setup` and drop `src_compile` completely.

So, in conclusion,  the current ebuild will work and leaving `+=` would have been fine, too.


:bomb:


@datahead8888 You say that like a cherry pick is more complicated than editing a text file. It's actually a really simple command, according to the documentation; just `git cherry-pick 7e86d969df234d971e64f746f45d6cdbdf8f8b45`.


+1
Absoluter Hammer, geniale Wirkung!


@dbrgn Patterns sind ja schon cool aber sind sie wirkliche so viel lesbarer als
```rust
fn is_leap_year(year: u16) -> bool {
    (year % 4 == 0) && (year % 100 != 0 || year % 400 == 0)
}
```
Wahrscheinlich spricht da aber der C/C++ Entwickler in mir ;)


@adamdecaf fyi


ideed, I'm confused too
in Demul we have this flags vise versa, and it works :)


Change for reference:
![image](https://cloud.githubusercontent.com/assets/206973/5826302/913badc6-a0a6-11e4-9ef4-6316c0cca475.png)


> things feel naturally

highly dubious grammar


Does this work for you? Since this commit I get `sp--pair-to-insert: Wrong type argument: symbolp, (property . pairs)` on every key press. When you look in the [dash.el source](https://github.com/magnars/dash.el/blob/master/dash.el#L1490-L1540), you see that, `-when-let` does not use `-let` but actually uses the normal `let`. So

```elisp
(-when-let ((p . v) (1 2 3))
  t)
```

is expanded to

```elisp
(let (((p . v) (1 2 3)))
  (when (p . v)
    t))
```

which does not look right at all, because

1. As far as I know `let` does not support destructuring
2. `(p . v)` is inserted in call position


Ah, good point, nvm.


I reopened WICKET-4551, see my proposal there.


Fix #11


Thanks!


> sound ~~look~~ like


close #11


I haven't done a cherry pick myself yet.

If there's a question of what breaks the release branch, though, I'd agree it's good practice to consult with Quintus.


this commit killed build https://github.com/ResurrectionRemix/kernel_lge_g3/commit/bf0b4605ae7a4e5651de08571b2a1fa876b97043


Ich habs jetzt mal vorläufig so übernommen - schaden kann es ja nicht.


maybe whats why TecmoWC wont work ?


This seems like something that could be solved in your system and not in the base data which is supposed to model the steps needed to fill out and submit the form. Seems awkward to have two click steps just because your particular system needs it to execute the steps.


Recreating view dependencies too


tried to install liblz4-tool but didnt work


I think, as we're learning, the idea of having data independent of a system that uses the data isn't really tenable, as they're too tied. Same situation as the no-js thing, in a sense.


Italian character set also contains a circumflex i.


This file is identical to [feature_extractors/tests/test_is_mainspace.py](https://github.com/halfak/Revision-Scoring/blob/b9aa2acd80bac076f8d6e5d1d8fa78f779ede48c/revscores/feature_extractors/tests/test_is_mainspace.py).


si jeje como pudieran pero ahora intentalo en JAVA :japanese_ogre:


@capitn: I do not see MySQL specific code in this commit, so I don't think it is actually related to issue #11 ?


Yeah, maybe; both the comments I wrote in the day and some notes from 2010
which I have just found indicate that you are right, with "01" meaning no
decompression and the compression flag being bit #1 of the 2-bits header (I
tend to use zero-based indexes, so all is coherent with your version). I
wonder how much time this have been wrong. :P

On Tue, Jan 20, 2015 at 10:21 PM, MetalliC <notifications@github.com> wrote:

> maybe whats why TecmoWC wont work ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mamedev/mame/commit/cd61a2736d8149f6640a5756b83caae55b7b07ae#commitcomment-9364170>
> .
>


Is this line correct?


Oh ok Jw cause it looked like a awesome ranks plugin to try.


> better oversight over these patterns

better insight into these patterns?
a better understanding of these patterns?


> as far as we know

huh?


Thanks for all that @ppat!  Things look great on my end, I'd just like a non-snapshot version to actually integrate.  Could you do a release of this?


@SebastianZ I know you're not part of the FWG anymore, but could you tell us (if you remember) why you added this line? Because that's causing issue #281.

Florent


Why not?


cool!


If I want to obtain all inner component instances, I need to get a list of all inputs and outputs of this component and then:

a) find all component instances, which have a data port instance, which is nestedBoundTo an input/output of the template
b) find all component instances bound to instances from a

In this case, it's not that bad, because the file is not large, but in a larger graph, it might take a longer time.
Could we add something like membership in case of pipeline <----> component instance?


Ah, take a look at the error here: http://stackoverflow.com/questions/24098429/widgethost-activity-to-fragment  This was what prompted not using the top 16 bits. Maybe the restriction is only for the compat library `Fragment`?


Also lets add attr( 'target', '_blank' )  to the href to work around any javascript issues against window.open ( window.open ) should be in a try catch.


You mean the Turtle syntax for multiple objects and same subject and predicate, the comma? That should be correct. But you have mentioned Jena having problems with this earlier I think


Fixes https://github.com/moneypot/vault/issues/2


I think messageblocks goes before myfab5...


恐らくRelationalの表現をしようとしていると思うので，`has_many`などを利用したほうがいいと思う．

http://www.stonedot.com/lecture6.html


Would be better to use a Docker fixture for this, no? Cf. https://github.com/jenkinsci/xvnc-plugin/pull/5.


extra close bracket?


I can't tell what's changed in this commit. A really neat way of exposing the change is to commit the upstream copy of the template first and then make your changes in another commit. Something to consider next time :)

Can you describe the change you've made?


I mean the fact that 1 input is bound twice to the same instance of an output.


Did this work or what?


typo: `pupose` -> `purpose`


Oh, now I see it, sorry.


Yes, I have thought about that. It is redundant, but we can have something like ldvm:nestedMember


Unclear what you mean by "locally" since you use it twice. Consider dropping the first use.


You gotta love this stuff!


eeew, how did i manage to write this in a header?


`Table 16: Texture Config Parameter 1`, my bad. :(

I heard you wanted some code review so I dig into the code and it's quite neat! I have no experience in driver writing but reading you code is quite pleasant considering we are talking about hardware related stuff.

I will certainly comment stuff here and there, do what you want, I'm not sure my comments are valuable as once again I have few hardware dev experience. Don't feel compelled to answer if it's not relevant.

Keep the good work!


Ya profe a ver si me quedo


I would recommend to open a PR right away and reference the ticket so that we prevent duplicate efforts and also others can help you more easily wrap this up. thanks for working on this!


Split of a branch


OK


Test


using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace Problema1
{
    class Program
    {
        static void Main(string[] args)
        {
            int[] numbers = new int[args.Length];

            Console.WriteLine("Number of command line parameters = {0}", args.Length);

            if (0 == args.Length)
            //testcase 1
            //////////////////////////////////////////////////
            {
                Console.WriteLine("Error: Null size array!");
                return;
            }
            /////////////////////////////////////////////////

            Console.WriteLine("Problema 1");
            //problema 1
            ////////////////////////////////////////////////////////////////////////
            int suma = 0;
            //testcase2
            //////////////////////////////////////////////////////////////////////////////////
            if (suma == null)
            {
                Console.WriteLine("Error: Variable suma not Initialized!");
            }
            //////////////////////////////////////////////////////////////////////////////////
            for (int i = 0; i < numbers.Length; i++)
            {
                numbers[i] = Convert.ToInt32(args[i].ToString());
                Console.WriteLine("numbers[{0}] = [{1}]", i, numbers[i]);
                if (numbers[i] % 2 == 0)
                {
                    suma = suma + numbers[i];
                    //testcase2
                    //////////////////////////////////////////////////////////////////////////////////
                    if (suma % 2 == 0)
                    {
                        Console.WriteLine("The number is even");
                    }
                    else
                        Console.WriteLine("The summ is not even one of the numbers is odd");
                    //////////////////////////////////////////////////////////////////////////////////

                }
                //testcase4
                //////////////////////////////////////////////////////////////////////////////////
                else
                {
                    Console.WriteLine("number not even");
                }
                //////////////////////////////////////////////////////////////////////////////////

                Console.WriteLine("Suma: {0}", suma);

            }
            Console.WriteLine("suma elementelor pare: {0}", suma);

        }
    }
}


Maybe you could add something about how this prevents multiple `Binary` instances being declared for `HashMap` (so long as everyone imports it from `Data.HashMap.Strict.Extended`). Not sure how important this is though.


True, if it could bump automatically every time we do a merge into master,
that would be the best.


Fine, I will add that and use it. Thanks.


The assertion could be more precise.


:top: :dancers:


@Lauricio Was there an issue with less 2.2.0?


@michaelwehrley I'm surprised you have to explicitly set the zone here. What a pain.


Changing the dependency file to add new dependencies from another file smells like a workaround. We should either just modify the existing config, or give mobilespec a way to dynamically accept a list of plugins to test. This needs more discussion.


It is related to `android.support.v4.app.FragmentActivity`:
http://grepcode.com/file/repository.grepcode.com/java/ext/com.google.android/android/4.4_r1/android/support/v4/app/FragmentActivity.java#836
It's not in the native part. So its inconsistent in the case that the `android.support.v4.app.Fragment` is not used there. But i still don't really get it whats the issue is. I use this: `IntentIntegrator(getActivity())`.


started cloning the source I will create a pull request when fixed and build successful ,browsing the code is more easy on my work station


Why?


I don't prefer this kind of white spaces. But this commit helped me see some trailing spaces to trim. So thanks for that.


Yes it works.  Get the latest version of dash.

If you run from MELPA, *always* update *all* dependencies manually because package.el *will never* do that for you if you use MELPA.


I think we can do:

```
val ordering: Ordering[HeavyHitter[_]] = Ordering.by[HeavyHitter[_], (Long, Int)] { hh => (hh.count, hh.item.hashCode()) }
```


"an" -> "a"


The 'str' parameter is unused and should be removed


PS: The magic is hidden in `dash--match`.


Thanks. :D


Just host.getCurrentDirectory is sufficient


[![Deploy to Azure](http://azuredeploy.net/deploybutton.png)](https://azuredeploy.net/)


> Having many modules is a great thing

Too strong? Having many modules is *fine*.


but no !!!
failed using github for windows, reversed my commit I don't know how !
All I want is to rename the commit as asked in https://github.com/ezsystems/ezpublish-legacy/pull/1137#issuecomment-70473364


Deleted the extra comma you pointed. Thanks.


Why make this one a function? Why not just store the bit on the program if it's not gonna change?


To my mind "proprietary" has bad connotations. Consider removing?


I am sceptical here as we still not have CI where to run those. At least on my box, ATH tests using docker causes more random failures than git and svn tests ever caused when integrating with actual repositories (before they switched to docker). It works locally and on CI server, I have no motivation to move it to docker.


host.getCurrentDirectory


Any chance these could be passed in through
```
-DARCH_DEFINES="-DTARGET_POSIX -DTARGET_LINUX -D_LINUX -fPIC"
```
when calling cmake like all the other platforms (at least all the `-DTARGET_FOO` are passed in for all the other platforms)?


Why? `s/==/=/g` should work fine, and better without bashisms.


Created a readme.txt


So, @nex3, given the slightly different semantics, is it problematic for you to have this rule use the same name as Sass's rule?  Or should we try out a slightly different name, so as not to collide?


Job scala-2.11.x-validate-main failed for 43289bc0, Took 0 s. (ping @adriaanm) [(results)](https://scala-ci.typesafe.com/job/scala-2.11.x-validate-main/66/):
If you suspect the failure was spurious, comment `PLS REBUILD 43289bc01bb8c5d7e09b8a5ea71b773c02f35bdb` on PR 18 to retry.
NOTE: New commits are rebuilt automatically as they appear. A forced rebuild is only necessary for transient failures.
`PLS REBUILD` without a sha will force a rebuild for all commits.


Deleted redundant 'r' variable. Thanks.


What does support incremental parsing?


I checked and there's no handling of dependencies inside dependencies so this probably only helps a standalone build of kodi-platform. But IMO it might be worth persuing that approach since it makes the dependency handling easier in that the addon does not have to know about the dependency of a dependency if the "first order" dependency defines its dependencies automatically.


I find it problematic to call global initialisations (static, user-independent) from an object instance that is user-dependant. With this new code, it is not possible anymore to have 2 FreshRSS objects, and it makes it difficult to change user. It creates problems such as https://github.com/FreshRSS/FreshRSS/issues/759


why 10?


Ok, thanks.


What is this version for?


You could also mention the idea of having a `Prelude.Extended` module to make common imports less of a hassle.


proprietary


Hmmm, where is the arabic translation ?


This method can be inlined now


parts of this commit breaks the rom boot. Sensor service can't access some directorys.


I'm using Grunt for automated tasks so I'm not planning to add pom.xml
The complexity chart you sent me was sweet though. I'll add a similar task to grunt in future.


@ericcf I did so b/c if you push to travis after 6pm it fails and I got sick and tired of it.  I believe this solved it; I have yet to push after 6pm however.


You're probably need to call the different jacoco goals to actually make it do something. http://www.eclemma.org/jacoco/trunk/doc/examples/build/pom.xml


Right. Is GitLab okay with accepting strings like `FIxes #` and `FIXES #`?


Thanks, you are right. Somehow my dash was outdated, even though I always update everything with `cask update`. Now there are no more error messages, but somehow pairs are still not closed automatically.


Modified


use [std::queue::empty()](http://en.cppreference.com/w/cpp/container/queue/empty) ;)


@PrzemekSzyszka @methyl What is the method you are using to extract this from squerb_widgets if I use `curl` or `wget` from `assets/backbone/application.js` I see a bunch of differences in newlines, some comments, and odd constants.


When moving paths, please adjust them in the CSS as well.


I prefer to continue with current directory structure because I think the file number will remain same. Thank you for the suggestion.


Refs #100


Fixed by 35cd863c50d4c5f9f279e48eaedc776811f86174


huh? it's not flink-graph anymore? :)


hum hum


Perhaps add:

> If they do get added, just remove the originals from `hackage-extended`, and the rest of your code doesn't even break!


@SubJunk I confirm that error. It causes an error in the DLNAMediaDatabase at the line 776. It seems to me that the method Script.execute was remowed.


What kind of pairs? If more things break I recommend downgrading to the latest tag version. In the very near future many more things are going to break... I'm doing quite agressive refactoring, and unfortunately melpa keeps distributing crap to users without me really wanting that.

Well, that's life.


> ... *a* great *side-effect* of ...


I think the only reason tmpdir is in there was to provide some options. It was first in the list to ensure that it matched the second entry, then got reversed such that the first directory always matched anyhow. I suppose this patch is okay given that it no longer relies on the state of directories not under the control of the person running the tests but maybe we could add `File::Spec->catdir(File::Spec->curdir, "blib")` to the front of MYPATH so that we can search for a "t" directory in two locations and search for a "t" file in two locations?


You could do that no matter where the functions live. This scheme just makes it trivial to separate them from the rest of your codebase.


> about why *a* particular function is useful.


ici, envoie plutôt une valeur qui a une sémantique plutôt qu'une valeur d'une touche


That's great! Much better :-) Small commits that add tidbits of functionality at a time. It's much easier to follow along with, and it gives me some insight into the way that you think about the project.

Does it make you feel better to make and push these small commits? I imagine it would!


Perhaps this would be a better title:

> Haskell Design Patterns: "Extended" Modules


Deleted boolean values from if conditions.
is.boolean and is.truthy checks will remain same.
Thank you.


What do you think of moving this class to your BrowserLocationView.swift? It's a pretty small class that I'd consider a part of the location view, and we could keep it private if we kept it there.


> the cssmenu.js already hides all the menus

This might result in the menus briefly appearing expanded before the script runs.


I [changed the paths](https://github.com/Tux64/bitcloud/commit/f59de9ec05d0df76e8e535843825176d3e302da3) to `wetube/bitcloud.c` and `wetube/bitcloud.js` after confirming that the links work between the modules.


Thanks for the heads up. Fixed this one.


I thought I did it, sorry


Looks good


Please use Array.prototype.forEach


@steveoh This mimetype config breaks my IIS for this project because I've already defined the same one at the server level. Since this is a config that would be shared by all of our apps eventually, I propose that we remove it from this project and apply it at the server level on all of our servers.

I would also add setting up this mime type as a set here: http://wiki.agrc.utah.gov/arcgis-server-setup-steps/

What do you think?


Here's hoping this wasn't happening much in practice.. :)


if (tx.time > 0) {e52b10499c8d00d3d332201c5ded2bf52cf621bf


Since this setting could vary depending on where the app is deployed, I would do something like:
```python
# use 'django.core.mail.backends.smtp.EmailBackend' if you really want to send emails
EMAIL_BACKEND = variable("EMAIL_BACKEND", default='django.core.mail.backends.console.EmailBackend')
```


You've got promises so you should return the promise, close the function and put the then by chaining to avoid callback imbrication


There is actually no need to check that we are in a chord and you can drop your changes. GetXRel gets the value from the aligner to which chord and notes point.


Why do you hate [me](http://www.slack.com)


Merged if statements this way. Thank you.


@Ichoran Thanks! Yes, that's the plan:
```scala
class C[@miniboxed T] {
  if (isMiniboxed[T])
    println("miniboxed")
  else
    println("generic")
}
```
Will produce:
```scala
trait C[T]
class C_L[T] extends C[T] {
  println("generic")
}
class C_J[T] extends C[T] {
  println("miniboxed")
}
```
This happens thanks to replacement in the Scala AST + the typer's constant folding.

> But I am really uneasy about assigning Boolean to Value. Very confusing when you see Boolean and don't know if it's a SimpleType#Value instance or a type. Yes, I know the compiler can keep it straight, but that doesn't mean I can. (And I need to also.)

What do you mean? Can you give me an example?


Hi @madarche. I think in this case `dependencies` is the place, because a user of the library (me) *will* need this indirect dependency to use the library; it's not something just for developers of convict. [Documentation](https://docs.npmjs.com/files/package.json#devdependencies) says:

> If someone is planning on downloading and using your module in their program, then they probably don't want or need to download and build the external test or documentation framework that you use. In this case, it's best to list these additional items in a devDependencies hash.


Hehe, yeah!  I just spent a good part of the weekend and holiday digging further into enproxy and finding/fixing more issues.  PR coming soon to a theater near you!


> But I am really uneasy about assigning Boolean to Value. Very confusing when you see Boolean and don't know if it's a SimpleType#Value instance or a type. Yes, I know the compiler can keep it straight, but that doesn't mean I can. (And I need to also.)

Oh, I get it now. Sorry, I'm a bit slow today. Okay, I'll come up with different names, maybe prefixed with Mb or something.


Actually, we don't even use this method, so for flashlight it wouldn't ever have happened in practice.  It's also a weird API, because whether or not hijacking is supported doesn't vary from request to request.  If it's not supported, the proxying would never work.


What was wrong with this?


it doesn't surprise me, the Naomi games run so badly in MAME nobody is really using them ;-)

I'll investigate things a bit further, thanks for the pointers hopefully we can get twc98 running properly too.


ok


4626-rhoCp_soil_calc_bug (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1586 of 1591 tests passed)

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


The string should be datatyped with ^^xsd:int


Idem with ^^xsd:dateTime


So for 'get all locations', what do we mean here? Get ALL ALL customer locations or get all locations for a given customer?


makeCacheMatrix <- function(x = matrix()) {
  inv <- NULL
  set <- function(y) {
    x <<- y
    inv <<- NULL
  }
  get <- function() x
  setinverse <- function(inverse) inv <<- inverse
  getinverse <- function() inv
  list(set=set, get=get, setinverse=setinverse, getinverse=getinverse)
}

cacheSolve <- function(x, ...) {
  inv <- x$getinverse()
  if(!is.null(inv)) {
    message("getting cached data.")
    return(inv)
  }
  data <- x$get()
  inv <- solve(data)
  x$setinverse(inv)
  inv
}


Yeah I am ok with that.


canola spelled wrong


You can report the error to the GitHub repository of morph-RDB


let's add new and do not delete existing.


But this is not a domain ontology, but the description of the R2RML triples maps


Variable milliseconds is unused here.


Shouldn't the district have its own URI?


This is what broke it last time


I think we should definately look at token based auth. A lot of people are making the switch for many good reasons. I read an article a while ago listing the pros and cons, ill have to find it but its worth a good read.


Profe no me dejo subir el proyecto pero le pongo el codigo de la clase abstracta


Dorian Fevrier <notifications@github.com> writes:

> `Table 16: Texture Config Parameter 1`, my bad. :(
>
> I heard you wanted some code review so I dig into the code and it's
> quite neat! I have no experience in driver writing but reading you
> code is quite pleasant considering we are talking about hardware
> related stuff.
>
> I will certainly comment stuff here and there, do what you want, I'm
> not sure my comments are valuable as once again I have few hardware
> dev experience. Don't feel compelled to answer if it's not relevant.

No worries!  Any comments are welcome, even if it's figuring out how
things work.


luckily I had it bookmarked :+1:

https://auth0.com/blog/2014/01/07/angularjs-authentication-with-cookies-vs-token/


Per @lukechurch's suggestion, this event is now triggered on the modification of the workspaces collection, piggy-backing on the existing DynamoModel events for WorkspaceAdded and WorkspaceRemoved.


4626-rhoCp_soil_calc_bug (mmatts) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


근데 아직 "구조.md" 가 보이지 않는구먼


At some point we removed the `Guid` property from `WorkspaceModel`. It is reintroduced here to uniquely identify `WorkspaceModel` objects.


Why don't you just remove the locale.conf and locale.gen from the other images?


Added tests to verify that the events are triggered and pass the correct workspace names.


Bien Josue! ya lo puede ver saludos!! ya cumpliste con tu primer tarea :+1:   @josueolvera


esta es cuando joven hereda los atributos de la clase abstracta


It crashed for unknown reason livecd script throwing error.


:+1:  this is crazy awesome!


Added curly braces to all one liner if conditions.


I think since it's not a default mime type for IIS and it's needed to make the app work properly then it should stay so a git checkout in a new windows vm will work. If a user has added the mime type then they should be able to delete this config. That's my two cents.


@novasr Looks like you need a patch in BISON before this will go through:
https://www.moosebuild.org/view_result/11368


Or just remove the layout like we do in Bootstrap :p


I wonder if we should prepend this with X (X-DumbsterAllRecipients) so it's clear it's a custom header.


testGetWithInvalidQueryReturnsBadRequest


\Clousure also callable - new \Clousure ?


Shouldn't the title be "System Administrator"?


I can see where you are coming from. The pain was that it took me a bunch of troubleshooting to figure out what was wrong. I'll just add a note to `README.md`.


Yeah, I'll end up switching to Sass for this I think in a future update. For now I wanted to minimize changes and just fix the errors from GitHub ;).


code style

```
}
else {
```


For consistency with the if test below, let's use equals() here: SmtpState.RCPT.equals(request.getState())


Sure thing. I'll try to make a PR tomorrow with some quick and easy fixes.


Bootstrap modal template from http://getbootstrap.com/javascript/#modals


Nice work ! Thanks for contributing :) Gonna review this


There were a couple typos in this commit message. Most importantly: "advise not to use a site name with a period in it," should read: "advise not to use a site name without a period in it,"


I guess the 's' shouldn't be at the end of the line.


If we want to match Android's behavior, you could also look for "m" or "mobile": http://mxr.mozilla.org/mozilla-central/source/toolkit/components/reader/AboutReader.jsm?force=1#556.


Make this a non-static function:
```java
public HashPagePartitionFunction withPartition(int partition)
```


Ideally, I think the dimensions should probably just be 1em and 1em (excluding or including the borders, both ways, I'm fine as long as 1em is set as height and width).


Tabs vs spaces.  The file I got given used tabs.  Ugh.

Fixed.


Another reason to jam this into a yeoman generator


I will file a bug.


2.0 actually ; see CHANGELOG.markdown
(might get confusing for people looking for version 2.0 otherwise...)


Thanks


implementacion de la clase joven


Your answer can never be <= 10 (in this case, 10) so you can start number from 10 (end of range)


That was actually holdover code from an older commit. Thanks for the catch!


this should be rebased on master right after wip-crush-straw2 merges.  otherwise looks ok!


or this can be 1 to match the constructor


Thanks Thomas!  I am probably going to have to do a hybrid of the two fixes.  I believe you are probably using the incl= correctly.... I will look it up and revise mine.

I think it might be even better to simply rely on the exact Apache root as specified in by the Apache parser.  That way if people would like to use AugeasConfigurator (that is not Apache) we don't have to load in unnecessary files.


What is <code>%READER-MESSAGE%</code> supposed to be used for?


This could feel like a hard assumption but as long as your range is bigger than 2, you can always increment with 2 instead of 1 because the answer has to be an even number.


What about [One link](https://www.slack.com) [and a ref link][id]

[id]: https://www.geekwire.com/


What's this authorize application business?


ISTM like these could be making assumptions about why the error occurred that aren't true, though perhaps it's completely uncommon to reference these traits in any other context than format strings.


Thanks for getting at this though... it is definitely an issue that should be addressed sooner rather than later!


i think this function can be dropped.. it's no differen than pg_to_up_osds, right?


I think you could eliminate the explicit copies with:
```python
    form = OptionalInfoForm(request.POST, instance=request.user)

    # This shouldn't happen since all fields on this form are optional.
    if not form.is_valid():
        return {
            'form': form
        }

    form.save()
```


You can just round these to 2.33em


'tasks' -> 'threads'


That's why I don't like excessive warnings - they make for kludgy code.


What about [two](http://slack.com) [links](https://slack.com/)


I reverted to the 1.6.3 tag for now and it works again.

Thumbs up for aggressive refactoring :)


Same, you can just round this to 0.417


The following will always return false. I think you meant to use || instead of &&:

biome == BiomeGenBase.forest && biome == BiomeGenBase.forestHills

You could do something like:

if(BiomeDictionary.isBiomeOfType(biome, BiomeDictionary.Type.FOREST && !BiomeDictionary.isBiomeOfType(biome, BiomeDictionary.Type.HOT &&
!BiomeDictionary.isBiomeOfType(biome, BiomeDictionary.Type.COLD &&
!BiomeDictionary.isBiomeOfType(biome, BiomeDictionary.Type.DENSE &&
!BiomeDictionary.isBiomeOfType(biome, BiomeDictionary.Type.SPARSE &&
biome != BiomeGenBase.birchForest && biome != birchForestHills) {

I think it should work for Biomes O Plenty and Vanilla.

I do not know what your overall intention is for the Divine Trees, but limiting to Forest/Forest Hills will make it very rare when you use it with a mod like Biomes O Plenty which adds so many biomes.


Same here


What about [two word](http://slack.com) [links](https://slack.com/)


If we want to be super clever, we can also check all properties of an object with reflection too. We'd need to be careful not to get into a recursive loop though.

Also, I don't think your check regarding internal classes is correct. Any class implementing Serializable should be considered serializable surely>


this is only useful if the crush rule has a linear bucket.  maybe adda  CrushWrapper method like ruleno_has_adaptive_seed(int ruleno) and only call this if the crush rule for the pool needs it.


Fix description.


Same


Ça plante sur ma base de donnée parce qu'il ne trouve pas la colonne "deleted_at". Il faudrait la mettre sur le seed je suppose.


What about [two word](http://slack.com)


Is that what you see when you click on the link? You may have to login to github first to see the content at that spot. That's a public recipe though so you shouldn't need to do anything.


This return type isn't being used anyway, so I just changed `consume()` and `consumeLine()` to return `void`.


You can also round this to 2.06em.


The comparison isn't need, so I just wend ahead and changed `isLocal` to be a `uint8_t`.


Same here for the rounding : 1.57em seems enough.


sent the 20th of Jan 2014, from github version


Is that a standard URL format?


> But I think having a no-IO version would be good too, because if someone wants to use a pure Haskell-diff function, one shouldn't have to touch the IO monad.

If you have a non-IO version, the IO version is just `(return .)` away. I know, it sometimes may feel like you're contaminating your code by wrapping it in IO, but that's only psychology.


<pre>[[ -f "/usr/bin/pulseaudio-ctl" ]] && /opt/livecd/pulseaudio-ctl-normal</pre>
... would have it done also ;)


Do you mind moving this into a `project/msvc2013` directory? I'll move the XCode stuff under `project` too when I get a chance.


Its OK if its not, I'm just curious. Because I'd tend to put the "scheme" before the `://`. Thoughts?


What about [two](http://slack.com) [word links](https://slack.com/) `some code`


> What's the advantage of having type-specific max/min functions instead of a generic macro?

Preventing double evaluation is what comes up right now, not sure about the other advantages but maybe I had something else in mind when I wrote these.


add `getRegion` to page


Do you get this when compiling as C, C++, or both?

Can you try changing the arrays in ObjClosure, ObjInstance, and ObjString from `[]` to `[0]` and see if that works?


nitpick: why is h1 capitalized?


Would love to see that happen. Just not sure how to maintain it.


Read the commit comment! (-:

I tried calling Jan to ask at JCMT as I thought he was TSS, but he had only
been there for daytime observing.   Will speak to him after volleyball.


This looks really nice. :+1:


What about [two word](http://slack.com) [links](https://slack.com/)


That's a lotta dupes lol.


Adding a namespace seems like a good idea, but "_firefox_ReaderMode" is kinda ugly. Any reason we can't just use "ReaderMode"? The only namespace collisions we should have is with our own user scripts, right? If so, the _firefox_ prefix seems superfluous.


You probably don't want to check an `Activity activity` is `instanceof Activity`


Perhaps --listen should take a URL? And I think we have some legacy ENV names, but that `STRONGLOOP_` is now preferred over `STRONGLOOP_`.


Any updates on this in the meantime?


nope, I tested in live session.

I don't know why it crashed the script.


looks good to me.


should we reference the previous research?


TeamCity Nimbus :: 0. Feature Branches [Build 0.0.0.0](http://twcbuild.cloudapp.net/viewLog.html?buildId=1112&buildTypeId=bt8) is now running


@iaavo Why are these setter removed here?!


What ever. As long as it works now ;)


Cool, those are good ideas. Testing them out now.


seems to work now ;)


"parks"


This commit provide surprisingly huge performance gains: rather than each object switching render targets in order to draw its shadow all the shadows are draw at once (so only one render target change).


I am having trouble creating a THIRD numerical outcome in the coin section....if I had a >100 else, it still gives the first answer because it's larger than 10....


The transcoding settings are not working with the new transcoding setup.
see also -> https://github.com/E2OpenPlugins/e2openplugin-OpenWebif/blob/master/plugin/controllers/transcoding.py

The solo2 have only
config.plugins.transcodingsetup.encoder.0.framerate
and
config.plugins.transcodingsetup.encoder.0.bitrate

the dou2 have
config.plugins.transcodingsetup.encoder.<0,1>.<...>
see -> https://github.com/E2OpenPlugins/e2openplugin-OpenWebif/blob/master/plugin/controllers/transcoding.py#L14


Thanks!


Hey world!


What do you think if you write this by yourself.
I mean if number is very large, say 2519 (which is not the answer) if you get 2519 % 3 > 0 for say 3, you are still forcing it to calculate % upto 10 which are useless. You should stop as soon as you get any non zero remainder.


Probably want to extract this at a later date. Perhaps into a 'corenet' pkg.


Hi Jixie, it seems that you did not set your email with "git config" command.
Please run "git config --global user.name "Jixie Zhang" and "git config --global user.email jixie@jlab.org" on any computer you are using.
And you may wanna to "git commit -a --amend" and then "git push -f" to override this commit.


Upps.
Sorry i have not seen the
if  machinebuild in ('inihdp', 'hd2400', 'et10000'):


Yeah, I think this should be addressed in `wren_common.h`. However, I would like Wren to do the "right" thing automatically when possible.  @MarcoLizza, can you try changing the block around line 25 in `wren_common.h` to this and see if it does the right thing:

```c
// If true, the VM's interpreter loop uses computed gotos. See this for more:
// http://gcc.gnu.org/onlinedocs/gcc-3.1.1/gcc/Labels-as-Values.html
// Enabling this speeds up the main dispatch loop a bit, but requires compiler
// support.
//
// Defaults to on if supported.
#ifndef WREN_COMPUTED_GOTO

#ifdef _MSC_VER
  // No computed gotos in Visual Studio.
  #define WREN_COMPUTED_GOTO 0
#else
  #define WREN_COMPUTED_GOTO 1
#endif

#endif
```


TeamCity Nimbus :: 0. Feature Branches [Build 0.0.0.0](http://twcbuild.cloudapp.net/viewLog.html?buildId=1112&buildTypeId=bt8) outcome was **SUCCESS**
Summary: Tests passed: 1444 Build time: 00:05:17


Which string functions is it yelling at you about? We can probably just use the safer alternatives.


What about [two word](http://slack.com) [links](https://slack.com/)


I am logged in to github.


Nope. User scripts are loaded globally. So any page that uses a top level variable named ReaderMode will run into trouble.


Minor grammar note:

  : The library version of libfabric in string form.  The data parameter is ignored.


To clarify my primary worry is the translation subsystem.


Looks like "referrer" is the correct spelling, but it may not be worth the pain of fixing it.


Done


Not my code.


On Android it is used to display error messages. The flow on Android is a bit different and I don't think we will actually use the reader-message element for that.


@timlinux so this is how we solve that strange problem ? Thanks Tim


:+1:

(I do like how terse returning a void call from a void fn is, though...)


To upload from hg-git when the bookmark isn't advancing, use `hg bookmark -f master`


This looks great. I am looking at working on some other items in the gem this week. If you want to make a pull request id be happy to merge this code into the main line of the gem.


What about [two](http://slack.com) [links](https://slack.com/)


Is there a particular motivation for this? Does this mean that Wren does compile as C with VS using the limited C99 support VS has?


Hi there ASA ka


`ChoiceField` seems like a better model to use. It will give you a dropdown widget automatically, including on the admin site.


Can you simplify this to:

```c
#if defined(_MSC_VER) && !defined(__cplusplus)
```

?


> Any class implementing Serializable should be considered serializable surely?

@GrahamCampbell I think that's what the current logic does. There's a test that should cover that. Is something missing?


I didn't make this change so I assume it was an artifact of running the tests.


better safe than sorry…


I definitely want to know what's going on here before this gets merged. A release target with no optimizations won't be what users expect!


@karan @filipekiss

Currently, as is, it runs into permissions issues on OSX and Linux (I don't have a Windows VM).


Will need to correct for Windows or Linux/OSX for forward or backwards slash


Done


PTAL


4626-rhoCp_soil_calc_bug (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (597 of 597 tests passed)

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


The documentation here is inconsistent in a few ways. The paragraph on the 'override' command reflects the previous design, not `--copy-local` and `--link-local`. The help docs haven't been extended to the 'default' or 'update' commands. Basically all three of these should have the same text, supporting '--copy-local', '--link-local' and '--use-package'.


saw approval from nickelization
at https://github.com/basho/clique/commit/55bd9f89dd8fdf958aff7451ad5690e49ad7beda


merging basho/clique/feature/cuttlefish-datatypes = 55bd9f89 into borshop-integration-45-feature/cuttlefish-datatypes


basho/clique/feature/cuttlefish-datatypes = 55bd9f89 merged ok, testing candidate = 45f19cf8


confirmed.

```
› echo -e "Myfab5\nMessageBlocks" |sort
MessageBlocks
Myfab5
```


Yea, EPEL 7 will be fixed as well. Are there problems with the others: F20, EPEL6, EPEL5? I can mention them on the bug report.


Why this change?

If you want to set remove iso default false, you just switch true false.


I'm lacking support for automatically registering ParamConverterProvider with AutoConfig. Would be great if you added it?


This function can be called from any of the 'update', 'default' or 'override' commands but the error text says 'override'.


Fixes #56


Only with CentOS 7, Fedora 20+21 and EPEL7.5.

For CentOS 6 I still use EPEL 6.8. Could I also use 7.5 there? And should I? The goal of the Docker tests I guess it to test the common configurations.


initial commit


This change breaks deployment to heroku. I had to use the previous version of this file to get up and running on heroku.


Hi Akira ! I was wondering if you had any luck making a piano-like app with Swift in Xcode. I have been looking around for help. Is this a project that you have been working on?


FYI, the orleans.codegen.cs file should not need to be checked in to source code control.
It should get regenerated automatically each time the project(s) get built.


Like Greg says, going to leave it like it is.  I made the mistake of changing recipe names once before and it caused some issues down the line.  Thanks though.  I will use your advice for naming future recipes.


ops, I'll update the docs! good catch


doh, will fix


must `consumedPageCount <= pages.size()`?  If so, add a check for that.


you can just do ```if not settings....```, this is more *pythonic*


Why did this need to change? Not questioning it, just curious :)


вот это я охуенный программист!


What about [two word](http://slack.com) [links](https://slack.com/)


You are using this in heroku? Wait, so you you're using heroku like a development environment? Could you please elaborate? I understood this script as the kind someone would run when hacking on strider locally.


Instead of passing the command line arguments through to `update_toolchain` would you mind doing the command line interpretation in the main script and passing 'true' or 'false' instead? Just a personal stylistic preference.


What does happen? Or what does not work? Do you have an example up somewhere?


arg.

last diff should read

```
...
queue = "test.queue"
...
```


Did this really get released? [This](http://updates.jenkins-ci.org/download/plugins/email-ext/) does not show it, and it is not on the update center. Did you do something other than `mvn -B release:prepare release:perform`?


This assertion has strayed a distance from the setting of $_toolchain. Should be immediately after.


Better to set the default to False, once we have a documentation we'll describe the setting


tem correspondência?


This used to not be necessary (nor is it necessary in other shells): the return value of the last command executed within a block becomes the function's return value. Is this an intentional change?


This interpretation of `$RETVAL` is far from the call that should have set it. Can you instead set a local to the value immediately after the call?


Oh, don't forget to add a small entry to changelog.html and of course add yourself to AUTHORS.rst


Also the [wiki](https://wiki.jenkins-ci.org/display/JENKINS/Email-ext+plugin) does not mention the release.


Yes


Instead of putting these into `MOCK_DIST_DIR` which is supposed to represent the same structure as the distribution server, can you put them into a new folder, perhaps `CUSTOM_TOOLCHAINS` (tmp/custom-toolchains)?


all tests pass:
success: http://buildbot.bos1/builders/test-clique/builds/33


What makes you think that the commentary about the `\` operator is anachronistic?


Also add tests for `--copy-local` and for 'default' and 'update'.


Hmmm, that's odd. It did complete successfully. I'll double check.


Where are you setting that variable?  With a "docker run -e=" ?


I need to install Nginx, HHVM FASTCGI and php5FPM along with SPDY module enabled on a cPanel server, nDeploy could handle all these tasks effectively ? Any benchmark results snapshots before/after using .Ndeploy using Nginx?

Thanks.


Why not use the `tempfile` module here?


Yeah, our strider is deployed onto heroku so we have a cloud instance listening to our github account for pushes. Then it runs the build on the cloud in heroku.

Is strider something that should just be a local thing? We were hoping to not have to set up a local machine for that. However, if it would cause less headaches, I can certainly make that happen.

Peter

> On Jan 20, 2015, at 4:08 PM, Keyvan Fatehi <notifications@github.com> wrote:
>
> You are using this in heroku? Wait, so you you're using heroku like a development environment? Could you please elaborate? I understood this script as the kind someone would run when hacking on strider locally.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/Strider-CD/strider/commit/d203219ed24f8e0ab23289425250ae092de2c651#commitcomment-9365792>.
>


Hm.. Du könntest `<text variable="container-title" font-style="italic"/>` bei Kapitel explizit hinschreiben (sozusagen eine Wiederholung der Zeile weiter unten), dann würde es gehen. Der Vorteil wäre, dass es einfacher ist, den Stil zu verstehen.

Wenn Du mit Deinen Überarbeitungen fertig bist, dann schreib bitte eine kurze Nachricht in Deinem Pull Request, da die anderen darüber informiert werden.


Haha, I feel like in the matrix. Bam, "I know threading!" hehe..

How is your stuff going?

Watching a new really cool series: Continuum. Recommended!

M

On Tuesday, January 20, 2015, Alexander Christie <notifications@github.com>
wrote:

> Looks like a pretty neat set of changes!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/serioth/FFXIV_Bot/commit/6b40fe1853b48b9cd12183e5859cfd9739de5281#commitcomment-9363263>
> .
>


Actually, there were a few cases where -Wunused-parameter made me think, so it's border line worth it.  But this is as far as I'll go.  Won't try to fix -Wswitch-default or -Wswitch-enum.


I really like what you're doing with the docs. Even when you're making my merges harder ...

 The pros/cons thing is good.


It looks like this is a duplicate test of the one right above it.


Small whitespace issue here. Additionally it looks like this code is directly duplicated below. Seems like a good thing to break out into a separate function.


I can't reproduce a problem with F20 in Python 2. But for some reason, I can't install the Python 3 basemap package to mock. Maybe you have the same, and basemap on Py3 is installed by `setup.py` instead?


Why is the substitution necessary?


@valib yeah I think we should remove it anyway. I see the error in NetBeans but it still compiles for me, weird.


This commit or some previous brakes the Rhythmbox playlist view.
This are my git bisect results:
Bad: 31cc1d5f4e3c3582cb9c321e0c1e993161d23153
Compiler errors: cc720ee8c2ac6952d6d288cd02cccd01dc63b3a5
Compiler errors: b20906ddea7acc7febd6735363eb00fb3585e973
Good: c13434a2771bcfad13a2b46b2b34c835d201aadb
Good: c0ed93df10c6faaed3ead9e026341296bb31ec35
@ywwg: Can you have a look? Thank you.


The glyph icon title looks like it got the wrong value, "Submit Feedback", instead of "Download x".


All the installation procedures are here:
https://github.com/krischer/obspy/tree/image_comparison/misc/docker_tests/base_images

I can try again tomorrow..maybe something got messed up...running the whole docker test suite including recreation of the base images takes quite a while...


ItIt  is the tuple that is emitted by the operator.


Where's this called from?


Hmm, was this here for block invalidation or something?

-[Unknown]


Please change (remove it's apostrophe): " by modifying it's Sources.def file" to " by modifying its Sources.def file" too - thanks


Well, I suppose I should note that your F20 dockerfile is installing an EPEL 7 RPM...


fast-forwarding develop to borshop-integration-45-feature/cuttlefish-datatypes = 45f19cf8


No need for logging here.


Just delete, don't comment out!


I think autologin group is only used by lightdm?
Do you know?


you can turn the `.apply()` into just `()`. there might even be a way to make this call apply() automagically, not sure yet, though.


TransOption (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2074 of 2190 tests passed)

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


It's not really important to have publishers.json up to date, but if you remember it, please update it whenever you add a new data directory.


spacing on {


Sorry :smile:


Fix the assignment.


so much easier to read this, nice


@henare thanks for the tip, I'll remember it for next time.

I've just modified lines 13/14 to say:
> You can find out how we use your email address on our [Privacy Page](https://www.righttoknow.org.au/help/privacy/)

It used to say:
> We will not reveal your email address to anybody unless you or the law tell us to ([details](https://www.righttoknow.org.au/help/privacy))


This change was unnecessary, because in contrast to the changes below it doesn’t change the outcome (one new merged object is created), but only worsens performance.


Hmm, what other format is there?

-[Unknown]


Remove print statements


Fix method name.


will do. I couldn't find a way to make the apply bet called automatically


You should probably change the upper boundary. We can't set more than 1px gap.


I honestly don't know, but it seemed to be having trouble creating the necessary virtualenv in python 3 (see the build errors in the failed builds), so I tried this and it worked. =P

On Wed, Jan 21, 2015 at 10:08 AM, Adrian Hecker <notifications@github.com>
wrote:

> Why did this need to change? Not questioning it, just curious :)
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jni/microscopium/commit/e8e3c10ef8e343c50d583c5fddc209580fd45985#commitcomment-9365789


I was thinking the very same thing at first (I always do the same for my projects). I'll Move the project and go for another pull request.


@bonniejools the return value of `get_or_create` isn't a `Model` instance: https://docs.djangoproject.com/en/1.7/ref/models/querysets/#get-or-create


More descriptive method name please.


Fixed


doesn't `.asSecrets` return an Option[Secrets]?


Ya está probado?


@bonniejools I don't really want to have commented out code committed if we can avoid it: either we need it or we don't...


Hey Peter,

That's great, but I was just referring to this file, "dev.sh" -- this should most certainly not be executed at all in deployment or on heroku... it's just a convenience script for `npm link`'ing the various other projects as git repositories so we can hack on multiple parts of strider as separate git repos easily while still having them in the npm dependency tree for strider.

That said why would this file "dev.sh" would have any effect on heroku ?


Both in C and C++. I'll give a try tomorrow.


What about [two word](http://slack.com) [links](https://slack.com/)


This return statement can be put in an `else` block so as to not catch `ImportErrors`  improperly.


Mostly "sprintf()", "strcpy()" and "fopen()". That is, every function getting a "char *" formal argument.


Me too, but I hate warnings. ;)


should this be a `val` inside the computation/closure?


Bordel Max qu'est-ce que c'est que ce nouveau projet secret ?


LGTM


Perl's Digest::Skein is the wrong version (1.2 while we use 1.3). I had to download SHA3 submission CD packages, until I found the matching test vectors to figure this out.  I hate hashes that have 6 different 'tweaked' versions, but only 1 name.  Whirlpool is another.  Fortunately, there is almost NO ONE using Whirlpool0 or Whirlpool1 (the first 2 versions).  But this one, there are people that have implemented at different stages, but all call their stuff skein.


TransOption (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg)


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/dabf7aaba9ff93ea2cd773ed33c4866357de1a42


merging basho/riak_cs/bugfix/suppress-unnecessary-warnings = dabf7aab into borshop-integration-1053-bugfix/suppress-unnecessary-warnings


basho/riak_cs/bugfix/suppress-unnecessary-warnings = dabf7aab merged ok, testing candidate = 7bb560ae


It was due to the unavailability of the "inline" modifier (but "_inline" exists) when compiling in plain C. I devised a solution, will provide a new pull request.


Aha, you spotted my deliberate mistake :-/ Fixed now.

On Tue, Jan 20, 2015 at 11:24 PM, Skud <notifications@github.com> wrote:

> Where's this called from?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Growstuff/growstuff/commit/aa72391b754c46c6c3730bdea3b04635ccd00fa7#commitcomment-9365985>
> .
>


Yeah. I'll fix this. I'll make it use the biome dict and just make sure it isn't a taiga, jungle, etc


Hi Helge, See you picked up on the plugins. It seems i mangled some callback limits (probably copied them from a version that was not correct).

Please note that I'm still thinking about changing the plugin structure a little to make it more flexible and easier to use.

Are you actually using the plugin version for testing? I'm always curious about any experiences with it (eiter positive or negative).

BTW: I got an error compiling the latest merbanan/rtl_433 on Raspberry Pi on uint8_t, had to include stdint.h, does this ring a bell?


... or rather, not fixed:
https://travis-ci.org/Growstuff/growstuff/builds/47718215


document timeout unit of measure


I'm working on it. It seems that the optimized goes ballistic when inlining the three successive "WREN_UNPIN" calls. As I wrote in the issue I opened, by selectively disable optimization around the WREN_PIN/UNPIN functions (not the macros, the called functions themselves).

I'll keep you posted.


Instead of having an individual sound command for each smote (sound-emote), a corresponding switch case in ClientCommandFactory.build(), ServerCommandFactory.build(), ClientWindow.playSound() and a file load procedure ClientWindow.loadSounds(), BitchTalk now loads and plays each sound dynamically.

All messages sent through Client are checked if it corresponds to a smote and is, if so, sent to the server with a tag to mark if it is a regular smote or admin-only smote.

Admin-only sound files are now denoted with an "admin_" tag in the front of the sound name.

Sound files that aren't designed to be used as smotes (notificationSound, userJoinedSound) are now denoted with an "other_" tag in the front of the sound name.

----------------------------------------------------------------------------------

As an example:

Client commands are built and executed as before.
Non-admin smoteare re-encoded: "/bossassbitch" becomes "/:s:bossassbitch" and is sent to the server.
Admin-only smoteare re-encoded: "/whatsgoingon" becomes "/:a:whatsgoingon" and is sent to the server.
Non-Client and non-sound commands are just sent straight to the server and are parsed in ServerCommandFactory.

====================================================

In ServerCommandFactory, if the recieved input isn't any of the normal server commands (Kick, SetName), it gets checked if it is a sound command before being discarded as NotACommand.
This check is perfomed by checking if the beginning sequence of characters is "/:s:" or "/:a:".
When checking for the "/:a:" tag, it simultaneously checks if the user is an admin.

It then strips the input of its tags, adds  "admin_" to the front of admin-only smotes and ships it off the the client, as to make sure the correct filename is used in ClientWindow.playSound().

Smotes Ahoy!


What about [two word](http://slack.com) [links](https://slack.com/)


I'm wondering if there is a way to move some of the `var` definitions to be inside the computation thus converting them to `val`. Let's talk


Nice.


OK, thanks for pointing out that it's just Symbol.@@toStringTag that's non-configurable.  I guess then that it is deliberate, with the reasoning being that in general things should be configurable but Symbol warrants extra protections.  If you're updating the note can you make it say something like that?  Thanks.


111 | ERROR | Doc comment for "$name" missing


Yep. Will do.


I wish I had seen your work before factoring out all the device code. Although, to be honest, I am not using the plugin code at the moment, but I saw the 5n1 issue when I tested it to see how it worked. It's definitely a nice framework, even though I would personally prefer a way to specify the plugins to use at runtime.

Yes, the missing include is a known issue and addressed in a pending pull request. I did all the refactoring on OS X, where my compiler seems to be more forgiving. I have now started to test on Centos 6 as well.


What about [word](http://slack.com) [links](https://slack.com/)


happy-1.19.4 add
```haskell
import Control.Applicative(Applicative(..))
```
to `Parser.hs`.  My recent patch uses `pure` in parser productions.
https://github.com/agda/agda/commit/9be9980cb9b534e8fa011ac7d31d1ae458e0216f

If requiring happy-1.19.4 is a problem, one could also add an import to Parser.y, hoping duplicate imports are fine.


minor: path and self.path diverge here.  not worth it if all you're trying to do is avoid using `self.`


All:

I agree that there could be a more surgical approach here.

I started out by putting in an assertion that src != dest and then looking at where the code broke.  My memory is hazy, but it was somewhere in array or domain assignment.  I looked at the Chapel code, and it did not appear to be easy to rewrite in a way that would avoid the src==dest issue.

My assumption at the time was that adding the test in the runtime code would not affect performance that much, because it was in a comm routine.  It would be negligible if communication were actually being performed.  However, where (src == dest) clearly src and dest are the same locale, so adding the extra "if" in front of the memcpy is noticeable.

I also thought that adding a special src == dest test in array assignment and potentially other places was uglier than handling the problem all in one place.  That said, there is plenty of C++ code for assignment that starts out:

T& T::operator=(const T& rhs) {
  if (&rhs == this) return *this;   // Don't do a deep copy if rhs and *this are the same object.
  ...
}

and sometimes the test is not merely there for performance reasons -- it is required for correctness.

SO, pushing the test into the module code is a reasonable elaboration of the initial commit.

Hopefully someone else can take on this effort....

THH
________________________________
From: Brad Chamberlain [notifications@github.com]
Sent: Monday, January 19, 2015 5:12 PM
To: chapel-lang/chapel
Cc: Tom Hildebrandt
Subject: Re: [chapel] Merge pull request #746 from hildeth/fixMemcpyOverlap (1745d1c)

@hildeth<https://github.com/hildeth> (also Cc:ing @gbtitus<https://github.com/gbtitus> and @ronawho<https://github.com/ronawho> since it has runtime implications):

Doing some detective-work on performance slips since the last release, this commit resulted in the ~15% execution time regression for the no-local version of the promoted op= test on 11/06, visible in this graph:

http://chapel.sourceforge.net/perf/chap04/?startdate=2014/09/18&enddate=2015/01/19&graphs=promotedoptimenolocal

The cause seems to be due to overheads added to comm calls contained within compiler-generated assignment routines.

Reviewing the commit, it feels like a big hammer in that, presumably, most calls to memcpy() are not made with the trivial src==dest case. To that end, I'm curious whether we could approach the valgrind errors that this silenced by addressing/fixing the code paths that did result in such calls rather than replacing memcpy() with chpl_memcpy() semi-globally.

For example, if such calls resulted from doing something like "A = A" (not that I'm saying they did), then I'd rather catch that case higher in the software stack than at the per-element assignment level. Or to say it's a user error.

Do you have any notes as to what code paths were leading to the valgrind issues?

—
Reply to this email directly or view it on GitHub<https://github.com/chapel-lang/chapel/commit/1745d1c84bb28521c2ac00494fe13247d44f3349#commitcomment-9348987>.


Why not use the result of `get_file_extension(fp.name)`?


Include the cause into the the new exception and remove the logging.


the tests should go in the unit_xxx.ml files. There is usually one in each directory.
Hardly a labyrinth.


Oh, never mind. Just saw the issue re: comments in CSVs


@RenaudR

Ligne 1829: Traduction directe. (Problème de cohérence) il faudrait dire quelque chose comme "Accéder au service theme" (theme étant le moteur) ou bien "Accéder aux services du thème" (Dans ce cas-ci, on ne sait pas s'il y a plus d'un service par thème ou bien un seul)

Ligne 1869: La c'est toi qui voit. Moi je trouve que "Échec lors de l'installation du thème" sonne mieux.

Ligne 1870: Problème de cohérence. Dans ce cas, il serait mieux de dire : "X a échoué l'installation." ou bien quelque chose du genre "X n'a pas pu être installé"


Nice!


i think `decode` should return an Option[Secrets]


I'll fix this.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/775


This can result in different instances for multiple threads, is it a problem in this case?


Interesting naming http://en.wikipedia.org/wiki/Immediately-invoked_function_expression

> 'use strict';

Should it be `"use strict";` instead? http://www.w3schools.com/js/js_strict.asp


In addition to the minor comments above, let's have the full review tonight.


Ok, thank you :)


Will fix.


Removed in commit c32e38c :)


just a suggestion, not a requirement: if we put the `if` into a new method like `def updateAvailable(mInx: Int): Boolean` into SecretData, and have `decode` return an Option[Secret], then this could look like this

```
if secrets.updateAvailable(currentModifyIndex)
  decode(secrets.Value)
else
  None
```


![screenshot002](https://cloud.githubusercontent.com/assets/6500800/5828680/85030ab6-a107-11e4-96f1-ea86a15cce45.png)
![screenshot003](https://cloud.githubusercontent.com/assets/6500800/5828681/8509f6a0-a107-11e4-8ce1-afb3eb5cf232.png)
Ok, only the video related widgets are done right now.  I'll do the audio and picture ones later.


What about [two word](http://slack.com) [links](https://slack.com/)


Works a treat! Thanks for that.

(I meant with the combine being under CP control, but thanks to Otterbear for clarifying.)


Sure, I don't expect this to be merged, but what I would like to do eventually is make a mode for salt that would show me user errors.


Whoops, missed that. Thanks.


Thanks, I should have known.


hello alan warren, i have been playing FFXI for the longest. i was wondering if you ever had a moment to explain some stuff for this website and things about gearswap. i currently have some sort of gearswap for all of the 22 jobs. but i am having problems with a few of them. like the corsair one is a little complicated and not doing things i want it to do. also i was wondering if there were some users that constantly keep there lua files up to date with the lastest best gear so i dont have to go asking and searching and number crunching to see what is the best for each situation for each job. hit me up if you have a moment. maybe we can chat on skype or something if you have a moment it might make things faster.


Looks like...nothing. I just got this from a `setup.py` that probably predates modern setuptools and I hadn't used setuptools enough to know that `setuptools.find_packages()` existed...
So, I'll switch this over to `setuptools.find_packages()` when I get a chance :smile:


There's no need for this, as the `requireUser` middleware will ensure that `req.user.uid` is always set :+1:


Don't know why I added it, maybe to visually separate blocks in disassembly view in MSVC, which doesn't really seem to be worth it.


" and ' are interchangeable in JavaScript, and I'm accustomed to using them over double quotes.

Yeah IIFE is a pretty interesting name - and I can never remember what the initial-ism stands for...


Eh, I had a bug and I wanted to avoid undefined behaviour. Maybe should remove this again..


Also check this out: http://www.w3fools.com/


This `tsv/csv` preprocessing should be happening in `csv_csv` and `pandas_csv` not in `render_html`.


Oops ;-)


I'm curious, @gordonwritescode how does `Command` end up in this scope if it's not required?


si


[![View of Koshigoe from Shichiri-ga-hama](http://api.artsmia.org/images/22594/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=22594)


[![GidayÅ« Chantress Reading Books](http://api.artsmia.org/images/22596/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=22596)


This constant should be at the top of the file, not in this function.


This is not the place to fix it... @targos


Something that didn't occur to me when I reviewed this is that at least when the gen compiler was gcc and it was optimizing, the memcpy() calls were probably being recognized and turned into inline copy loops. (At least doing "info gcc" and searching for "memcpy" indicates this would be so.) Replacing such inline loops with function calls to chpl_memcpy() could definitely be observable in performance, if there were many such calls.


What about [two word](http://slack.com) [links](https://slack.com/)


Olá,

Estou adicionando um readme ao projeto.


Why convert stack to array? Also, why not use toArray function?


Corrigindo alguns detalhes como o link para inscrição e o número de convidados.


@joebubna I'm getting these errors because these files are not checked in I think:

1>------ Build started: Project: Copiosis_Application, Configuration: Debug Any CPU ------
1>CSC : error CS2001: Source file 'Controllers\AdminController.cs' could not be found
1>CSC : error CS2001: Source file 'Models\AdminModels.cs' could not be found
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========


@eslaurente I have let @joebubna know about this. For now, do what you can without needing to build as adding your own files for these will just make your life crazy with the merge.


@kylegalbraith gotcha


Actually found a few bugs in the build system while looking at this! Fix is 893c0e9b6


This is the code before commit 186b7899ce7195bac0d075f1a5cdf56c74164db4 which has a bug.


Thanks!


From crosswalk master branch, it's   " [Custom] any getContext([Default=Undefined] optional DOMString contextId); ".  I guess your reference code is from upstream master,  I would like to remove it in future rebase.


reert


Why did you make the department_id unique here?


I'm not sure why tests are failing on your machine - they work on mine. In any case, this change seems to be a NOP, because the `count` method should be doing the same thing as what you intend. For example:

```
irb(main):002:0> (1..10).count { |x| x >= 8 }
=> 3
```


@mcm69 please [review](https://github.com/Animatron/player/blob/d2d7589a25d51752cceb981d38e5d2aafd48065a/doc/sequences.md) for grammatics.


One possible workflow in this scenario could've been to open a pull request against my fork.


Let us start it